[Sugar-devel] [PATCH 15/21 sugar-toolkit] pylint cleanup: disable warnings for reasonable catch-all exception handlers

Simon Schampijer simon at schampijer.de
Fri Oct 29 06:58:00 EDT 2010


Reviewed-by: Simon Schampijer simon at laptop.org

On 10/15/2010 11:01 PM, Sascha Silbe wrote:
> Signed-off-by: Sascha Silbe<sascha-pgp at silbe.org>
>
> diff --git a/src/sugar/activity/activity.py b/src/sugar/activity/activity.py
> index 64bc840..7229096 100644
> --- a/src/sugar/activity/activity.py
> +++ b/src/sugar/activity/activity.py
> @@ -823,6 +823,7 @@ class Activity(Window, gtk.Container):
>               try:
>                   self.save()
>               except:
> +                # pylint: disable=W0702
>                   logging.exception('Error saving activity object to datastore')
>                   self._show_keep_failed_dialog()
>                   return False
> diff --git a/src/sugar/util.py b/src/sugar/util.py
> index e0a431b..df56946 100644
> --- a/src/sugar/util.py
> +++ b/src/sugar/util.py
> @@ -335,6 +335,7 @@ def _cleanup_temp_files():
>           try:
>               os.unlink(path)
>           except:
> +            # pylint: disable=W0702
>               logging.exception('Exception occured in _cleanup_temp_files')
>
>   atexit.register(_cleanup_temp_files)
> --
> 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