Ok I just pushed the new version. I designed it by using exceptions. Is that pretty? ^_^
On Fri, 2012-02-24 at 16:26 +0000, Olivier Dony (OpenERP) wrote: > Of course, renaming `o` to `option` would be better too, because it is not an > integer index for a loop. Granted, the loop is only 4 lines so the meaning of > "o" and "e" are obvious enough, but code evolves over time and a 4-line loop > may become much more complicated after a while. In general we just don't want > to be forced to read to whole code to see what a variable stands names, so > this is a best practice to adopt from the start and use in all circumstances > :-) -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-environment-vars-cto/+merge/94564 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-environment-vars-cto. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
