Thanks for the explanation.

For a stylistic point: I would prefer to keep the line
  newtext = utils._process_text(self, node.text or '')
IMO it is easier to read that reusing a third time the same variable... 
(doesn't matter much I guess).

I will merge it, thanks.


-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-640677-nco/+merge/61564
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/trunk-bug-640677-nco.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-web
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-web
More help   : https://help.launchpad.net/ListHelp

Reply via email to