Code review comment for lp:~openerp-dev/openobject-server/7.0-rstcnx-chs

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Florent, thanks for spotting the undesired side-effect! I prefer your solution of removing that commit rather than switching to reverse_enumerate(), which could have other side-effects (e.g. like keeping dead connections in the pool for a longer time). That part was reverted at revision 4849 rev-id: <email address hidden>.

« Back to merge proposal