Review Request: EmergeBase: Ensure that an object of this type always has an attribute buildSystemType. Where it was nonexistant till now, it would now be None

Wolfgang Rohdewald wolfgang at rohdewald.de
Mon Jan 17 00:28:22 CET 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6345/
-----------------------------------------------------------

Review request for kde-windows.


Summary
-------

    EmergeBase.__init__: if the object does not yet have attribute
    buildSystemType, set it to None. This makes referencing this
    attribute easier because "if hasattr()" is no longer needed. And
    it makes pylint happy.
    The same thing has already been done with subinfo, r1214441, which
    resulted in no complaints or problems


Diffs
-----

  /trunk/kdesupport/emerge/bin/EmergeBase.py 1214807 
  /trunk/kdesupport/emerge/bin/Package/PackageBase.py 1214807 
  /trunk/kdesupport/emerge/bin/Packager/NullsoftInstallerPackager.py 1214807 
  /trunk/kdesupport/emerge/bin/Source/ArchiveSource.py 1214807 
  /trunk/kdesupport/emerge/bin/Source/FileSource.py 1214807 
  /trunk/kdesupport/emerge/bin/Source/SourceBase.py 1214807 

Diff: http://svn.reviewboard.kde.org/r/6345/diff


Testing
-------


Thanks,

Wolfgang

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-windows/attachments/20110116/717f5df3/attachment.htm 


More information about the Kde-windows mailing list