On Tue, Nov 17, 2020 at 12:28:27 +0100, Michal Privoznik wrote:
> Similarly to previous commits, we can utilize domCaps to check if
> graphics type is supported.
>
> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> ---
> src/qemu/qemu_capabilities.c | 2 +-
> src/qemu/qemu_capabilities.h | 3 +++
> src/qemu/qemu_validate.c | 40 ++++++++++++------------------------
> 3 files changed, 17 insertions(+), 28 deletions(-)
[...]
> @@ -3903,15 +3892,12 @@ qemuValidateDomainDeviceDefGraphics(const virDomainGraphicsDef *graphics,
> }
>
> break;
> +
> + case VIR_DOMAIN_GRAPHICS_TYPE_VNC:
> case VIR_DOMAIN_GRAPHICS_TYPE_RDP:
> case VIR_DOMAIN_GRAPHICS_TYPE_DESKTOP:
> - virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
> - _("unsupported graphics type '%s'"),
> - virDomainGraphicsTypeToString(graphics->type));
> - return -1;
> case VIR_DOMAIN_GRAPHICS_TYPE_LAST:
> - default:
> - return -1;
> + break;
Removing 'default: ' is not necessary once you use proper type for the
variable in the switch statement, which is our usual approach.
The default and _LAST case should use virReportEnumRangeError.
