Just to say I support Virgil on #2, and sparse better then dense. In my experience this also makes not only for better reading and comprehension, but also for much easier debugging.
And I do not see the point of adding return True to functions where this does not have any meaning: if a function contains (or detects) errors, it should throw an exception. No exception thrown means function did the work it was supposed to do. -- https://code.launchpad.net/~savoirfairelinux-openerp/banking-addons/loose-coupling/+merge/185033 Your team Savoir-faire Linux' OpenERP is subscribed to branch lp:~savoirfairelinux-openerp/banking-addons/loose-coupling. -- Mailing list: https://launchpad.net/~savoirfairelinux-openerp Post to : [email protected] Unsubscribe : https://launchpad.net/~savoirfairelinux-openerp More help : https://help.launchpad.net/ListHelp
