[Sugar-devel] [PATCH 20/21 sugar-toolkit] pylint: sugar.wm._property_change_trapped: allow overriding built-in method

Simon Schampijer simon at schampijer.de
Fri Oct 29 07:02:46 EDT 2010


Reviewed-by: Simon Schampijer simon at laptop.org

On 10/15/2010 11:01 PM, Sascha Silbe wrote:
> "format" is the name of a parameter of the function we are wrapping, so we
> shouldn't change it.
>
> Signed-off-by: Sascha Silbe<sascha-pgp at silbe.org>
>
> diff --git a/src/sugar/wm.py b/src/sugar/wm.py
> index 00c6fd0..8f1903f 100644
> --- a/src/sugar/wm.py
> +++ b/src/sugar/wm.py
> @@ -38,6 +38,7 @@ def _property_get_trapped(window, prop, prop_type):
>
>
>   def _property_change_trapped(window, prop, prop_type, format, mode, data):
> +    # pylint: disable=W0622
>       gtk.gdk.error_trap_push()
>
>       window.property_change(prop, prop_type, format, mode, data)
> --
> 1.7.1
>
> _______________________________________________
> Sugar-devel mailing list
> Sugar-devel at lists.sugarlabs.org
> http://lists.sugarlabs.org/listinfo/sugar-devel
>



More information about the Sugar-devel mailing list