Code review comment for lp:~songofacandy/bzr/i18n

Revision history for this message
methane (songofacandy) wrote :

> > - in bzrlib/utextwrap.py, you have tests protected by if __name == 'main',
> better put that in a proper file under bzrlib/tests
> > - in the same file you use width=70, how does that interact with th rest of
> bzrlib (osutils.terminal_width for example) ?
> I'll do these soon.
>

Done. I've request merge in:
https://code.launchpad.net/~songofacandy/bzr/i18n-utextwrap/+merge/59950

« Back to merge proposal