|
From: | Paolo Bonzini |
Subject: | Re: [Qemu-devel] [PATCH v6 8/8] qmp-event: add event notification for memory hot unplug error |
Date: | Mon, 13 Apr 2015 16:43:18 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
On 13/04/2015 16:40, Eric Blake wrote: >>>> +## >>>> >>> +# @MEM_UNPLUG_ERROR >>>> >>> +# >>>> >>> +# Emitted when memory hot unplug error occurs. >>>> >>> +# >>>> >>> +# @device: device name >>>> >>> +# >>>> >>> +# @msg: Informative message >>> >> Any reason you abbreviated instead of spelling it out as 'message'? >> > >> > This only refer to the spelling of event BLOCK_IMAGE_CORRUPTED >> > in docs/qmp/qmp-events.txt. > Just because existing code abbreviated does not mean that new code needs > to copy the bad example. To some extent it does; consistency is a good thing to have. So I think "msg" is okay here. Paolo
[Prev in Thread] | Current Thread | [Next in Thread] |