Re: [Qemu-devel] [PATCH 06/12] monitor: remove mhandler.cmd_new

[Eric Blake]

On 06/24/2016 08:20 AM, Marc-André Lureau wrote:
> Hi
> 
> On Fri, Jun 24, 2016 at 6:39 AM, Eric Blake <address@hidden> wrote:
>> On 06/22/2016 06:08 PM, address@hidden wrote:
>>> From: Marc-André Lureau <address@hidden>
>>>
>>> This is no longer necessary, now that middle mode has been removed.
>>>
>>> Signed-off-by: Marc-André Lureau <address@hidden>
>>> ---
>>>  docs/writing-qmp-commands.txt |   8 +-
>>>  hmp-commands-info.hx          | 118 ++++++++++++------------
>>>  hmp-commands.hx               | 206 
>>> +++++++++++++++++++++---------------------
>>>  monitor.c                     |  11 +--
>>>  4 files changed, 168 insertions(+), 175 deletions(-)
>>
>> The changes to hmp-commands*.hx are mechanical; you may want to set up
>> git to order the diff so that more relevant files like monitor.c are
>> listed first.  Do that by 'git config diff.orderFile /path/to/file',
>> then in that file, listing globs in the order that you want to prioritize.
>>
> 
> Nice tip, didn't know about it. I wonder if it would be worth to have
> one in qemu source tree..

Maybe not: it still requires a local setting to be useful (the file
existing in-tree doesn't mean that 'git diff' will automatically know to
use it without users editing .git/config); having it version-controlled
means that the diff you list now may depend on the version of the file
in your tree now rather than the version at the time the patch was first
generated for some confusing results; and there are still cases where I
personally find that I don't necessarily want the same order as last
time (so I'm frequently tweaking my own order file according to the
series I'm currently posting).

Hmm - maybe it should get a mention on the SubmitAPatch wiki page, though.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

signature.asc
Description: OpenPGP digital signature