[libvirt] [PATCH v2 2/2] conf: Parse guestfwd channel device info again
Andrea Bolognani
abologna at redhat.com
Tue Aug 21 11:18:59 UTC 2018
On Tue, 2018-08-21 at 12:12 +0200, Michal Privoznik wrote:
[...]
> if (def->deviceType == VIR_DOMAIN_CHR_DEVICE_TYPE_CHANNEL &&
> - def->targetType == VIR_DOMAIN_CHR_CHANNEL_TARGET_TYPE_GUESTFWD) {
> - VIR_DEBUG("Ignoring device address for gustfwd channel");
> - } else if (virDomainDeviceInfoParseXML(xmlopt, node,
> - &def->info, flags) < 0) {
> - goto error;
> - }
> -
> + def->targetType == VIR_DOMAIN_CHR_CHANNEL_TARGET_TYPE_GUESTFWD)
> + virDomainDeviceInfoClearAddress(&def->info);
Our style guidelines require curly braces around the if body in this
case - which is great, because it means your diff can be smaller and
more readable! :)
With that fixed
Reviewed-by: Andrea Bolognani <abologna at redhat.com>
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list