[libvirt] [PATCH 05/17] qemu_hotplug: cleanup qemuDomainChangeGraphics
Peter Krempa
pkrempa at redhat.com
Fri May 6 12:30:35 UTC 2016
On Thu, May 05, 2016 at 18:20:24 +0200, Pavel Hrdina wrote:
In subject: This mostly improves error messages, so cleanup is not
really a spot-on description of this patch.
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
> src/qemu/qemu_hotplug.c | 33 +++++++++++++++++----------------
> 1 file changed, 17 insertions(+), 16 deletions(-)
>
> diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
> index 03e5309..97f4152 100644
> --- a/src/qemu/qemu_hotplug.c
> +++ b/src/qemu/qemu_hotplug.c
> @@ -2618,8 +2619,9 @@ qemuDomainChangeGraphics(virQEMUDriverPtr driver,
> }
>
> if (dev->nListens != olddev->nListens) {
> - virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
> - _("cannot change the number of listen addresses"));
> + virReportError(VIR_ERR_OPERATION_UNSUPPORTED,
> + _("cannot change the number of listen addresses "
> + "on '%s' graphics"), type);
I'm not quite sure whether the 'on' preposition is correct in this
context, but I don't have a better suggestion.
> goto cleanup;
> }
>
> @@ -2628,30 +2630,30 @@ qemuDomainChangeGraphics(virQEMUDriverPtr driver,
> virDomainGraphicsListenDefPtr oldlisten = &olddev->listens[i];
>
> if (newlisten->type != oldlisten->type) {
> - virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
> - _("cannot change the type of listen address"));
> + virReportError(VIR_ERR_OPERATION_UNSUPPORTED,
> + _("cannot change the type of listen address "
> + "on '%s' graphics"), type);
> goto cleanup;
> }
>
> switch ((virDomainGraphicsListenType) newlisten->type) {
> case VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_ADDRESS:
> if (STRNEQ_NULLABLE(newlisten->address, oldlisten->address)) {
> - virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
> - dev->type == VIR_DOMAIN_GRAPHICS_TYPE_VNC ?
> - _("cannot change listen address setting on vnc graphics") :
> - _("cannot change listen address setting on spice graphics"));
> + virReportError(VIR_ERR_OPERATION_UNSUPPORTED,
> + _("cannot change listen address setting "
> + "on '%s' graphics"), type);
> goto cleanup;
> }
> - break;
>
> + break;
> case VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_NETWORK:
I prefer the empty line before the next 'case' statement.
> if (STRNEQ_NULLABLE(newlisten->network, oldlisten->network)) {
> - virReportError(VIR_ERR_OPERATION_UNSUPPORTED, "%s",
> - dev->type == VIR_DOMAIN_GRAPHICS_TYPE_VNC ?
> - _("cannot change listen network setting on vnc graphics") :
> - _("cannot change listen network setting on spice graphics"));
... ah so it was pre-existing. I don't care about it then :)
> + virReportError(VIR_ERR_OPERATION_UNSUPPORTED,
> + _("cannot change listen address setting "
> + "on '%s' graphics"), type);
> goto cleanup;
> }
> +
> break;
>
> case VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_NONE:
ACK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160506/ab38188b/attachment-0001.sig>
More information about the libvir-list
mailing list