[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