Code review comment for lp:~therp-nl/ocb-server/7.0_lp1260743_print_empty_lines

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks! This actually improves on https://code.launchpad.net/~olivier-laurent/openobject-server/6.1-bug1157048/+merge/154031, which either only works in other cases or has only worked before.

I'm hesitant about the changes in whitespace. I know that the OpenERP core dev team does not like unnecessary changes, so to increase the chances of this change being merged in upstream, I would like to ask you to remove them.

review: Needs Fixing

« Back to merge proposal