Looks good.

--Justin


On Aug 24, 2011, at 5:15 PM, Ben Pfaff wrote:

> There's no variable table_name.
> 
> Found by pychecker.
> ---
> python/ovs/db/idl.py |    4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/python/ovs/db/idl.py b/python/ovs/db/idl.py
> index 8f3c810..65f6838 100644
> --- a/python/ovs/db/idl.py
> +++ b/python/ovs/db/idl.py
> @@ -249,7 +249,7 @@ class Idl:
>                 changed = True
>                 # XXX rate-limit
>                 logging.warning("cannot modify missing row %s in table %s"
> -                                % (uuid, table_name))
> +                                % (uuid, table.name))
>             if self.__modify_row(table, row, new):
>                 changed = True
>         return changed
> @@ -269,7 +269,7 @@ class Idl:
>             except error.Error, e:
>                 # XXX rate-limit
>                 logging.warning("error parsing column %s in table %s: %s"
> -                                % (column_name, table_name, e))
> +                                % (column_name, table.name, e))
>                 continue
> 
>             if datum != getattr(row, column_name):
> -- 
> 1.7.4.4
> 
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev

_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to