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
