Re: [PATCH v3 17/23] monitor: Rename MonitorQMP @mon -> @qmon

Philippe Mathieu-Daudé <[email protected]>
Newsgroups gmane.comp.emulators.qemu
Message-ID <[email protected]>
On 13/8/26 11:26, Daniel P. Berrangé wrote:
> On Wed, Aug 12, 2026 at 11:17:01PM +0200, Philippe Mathieu-Daudé wrote:
>> Mechanical change to sanitize using the following patterns:
>>
>>    MonitorQMP *qmon  (this patch)
>>    MonitorHMP *hmon  (following one)
>>    Monitor *mon
>>
>> Rename @mon (and @qmp_mon) as @qmon when the type is MonitorQMP.
> 
> The more suitable names to me feel like  'mon', 'qmp' and 'hmp',
> not least because we already use them, whereas as have no use
> of qmon/hmon currently.

Interesting, this is what I chose first, then thought I'd receive
comments that this the monitor state being named the same as the
interface used would be confusing. I'll happily redo the rename
(but after Marc-André 'Make HMP optional' series lands).
> 
> 
>>
>> Signed-off-by: Philippe Mathieu-Daudé <[email protected]>
>> ---
>>   monitor/monitor-internal.h |   4 +-
>>   monitor/qmp-cmds-control.c |  18 +--
>>   monitor/qmp.c              | 256 ++++++++++++++++++-------------------
>>   3 files changed, 139 insertions(+), 139 deletions(-)
> 
> With regards,
> Daniel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.