[PATCH] Suppress errors in Python AppletScripts
Thomas Olsen
tanghus at gmail.com
Thu Jul 29 09:16:39 CEST 2010
On Wednesday 28 July 2010 01:18:22 Aaron J. Seigo wrote:
> On July 27, 2010, Luca Beltrame wrote:
> > Let me know if it's OK.
>
> in the patch, is
>
> +
> PythonAppletScript.importer.unregister_top_level(self.pluginName)
>
> the proper indentation for that line? (not a pythonista, so not sure if
> what looks like extra indentation there will matter any.
>
> otherwise, looks fine.
To me it looks like there's a mix of using tabs and spaces for indentation in
the patch. That can be seriously confusing when trying to debug ;-)
On a completely different and OT track: is the __dtor__() method a construction
added by sip? AFAIRC it's not from standard Python where one would use
__del__() instead.
--
Best Regards / Med venlig hilsen
Thomas Olsen
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 230 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20100729/3b96a536/attachment.sig
More information about the Plasma-devel
mailing list