On Wed, 11 Sep 2013 16:13:32 -0600
Eric Blake <ebl...@redhat.com> wrote:

> On 09/11/2013 04:08 PM, Eric Blake wrote:
> > On 09/11/2013 02:52 PM, Luiz Capitulino wrote:
> >> Populate it with all scripts stored in QMP/.
> >>
> >> Signed-off-by: Luiz Capitulino <lcapitul...@redhat.com>
> 
> > Reviewed-by: Eric Blake <ebl...@redhat.com>
> > 
> > Repeat after me:
> > git config diff.renames true
> > 
> > at which point, this commit is much more compact and reviewable:

Done.

> Oh, and while testing your patch:
> 
> <stdin>:1854: trailing whitespace.
> 
> <stdin>:2054: trailing whitespace.
> #
> warning: 2 lines add whitespace errors.
> 
> One instance from qmp-shell; the other from qmp.py.  It's okay if you
> touch those up as part of this rename (again, the diff.renames true bit
> will highlight those two changes, rather than showing the complete file
> as a delete and addition).

Fixed.

Reply via email to