Code review comment for lp:~jls-boost/bzr/bug952132

Revision history for this message
Martin Packman (gz) wrote :

Sorry for the delay reviewing this. Basically I don't think the added code complication is worth it for a corner case like this. Instead we probably want to do less magic with the commandline in the first place.

I've pulled out your typo fixes and sent them for merging separately however:

<https://code.launchpad.net/~gz/bzr/2.5_jls-boost_typo_fix/+merge/117156>

review: Disapprove

« Back to merge proposal