> * Did the customer provide the patch himself? in the email above, or was the
> patch re-done from scratch?
> * If the patch was the customer's, he should be credited (by using —author)
No, customer didn't provide any patch. Its done from scratch.
> * The "{0,1}" RE pattern can (and should) be replaced with "?".
> * Shouldn't we add support for a positive prefix just in case?
For this i have committed changes, no need to check "{0,1}" in re.
Please check and if anything missing tell me.
--
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-4396-sma/+merge/53227
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-client-web/6.0-opw-4396-sma.
_______________________________________________
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
