Nathan Bush <nathan.bush at sun.com> writes: > Hi, > > I need a code review for: > 423 questionable format for rename-only changes in hg webrev > > The change to webrev is: > http://cr.opensolaris.org/~nbush/scm-migration/423/webrev/ >
The only thing that jumps out is a nit I don't really mind either way: 2584: -n is perhaps better than ! -z The results also look ok, to me. -- Rich
