Re: [PATCH 1/3] hmp: remove "change vnc TARGET" command

[Eric Blake]

On 1/20/21 8:42 AM, Paolo Bonzini wrote:
> The HMP command \"change vnc TARGET\" is messy:
> 
> - it takes an ugly shortcut to determine if the option has an "id",
> with incorrect results if "id=" is not preceded by an unescaped
> comma.
> 
> - it deletes the existing QemuOpts and does not try to rollback
> if the parsing fails (which is not causing problems, but only due to
> how VNC options are parsed)
> 
> - because it uses the same parsing function as "-vnc", it forces
> the latter to not support "-vnc help".
> 
> On top of this, it uses a deprecated QMP command, thus getting in
> the way of removing the QMP command.  Since the usecase for the
> command is not clear, just remove it and send "change vnc password"
> directly to the QMP "change-vnc-password" command.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  docs/system/removed-features.rst | 6 ++++++
>  hmp-commands.hx                  | 6 ------
>  monitor/hmp-cmds.c               | 7 +++++--
>  3 files changed, 11 insertions(+), 8 deletions(-)

HMP is not promised to be stable, so no deprecation period required.
Reviewed-by: Eric Blake <eblake@redhat.com>

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org