Re: [PATCH 5/8] conf: domain: add VIDEO_TYPE_DEFAULT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 06/28/2017 02:49 PM, Cole Robinson wrote:
> Will be needed for future patches to pull the default video type
> setting out of XML parsing routines.
> 
> Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
> ---
>  src/conf/domain_conf.c              | 1 +
>  src/conf/domain_conf.h              | 1 +
>  src/qemu/qemu_command.c             | 3 +++
>  src/qemu/qemu_domain.c              | 1 +
>  src/qemu/qemu_domain_address.c      | 1 +
>  tests/domaincapsschemadata/full.xml | 1 +
>  6 files changed, 8 insertions(+)
> 

Well, I see why you added patch 3 now...  But I'm still convinced it's
better to have all switch/cases handle every case.

My only concern here would be related to migration, but since it doesn't
appear that "default" can be saved to the XML file I think we're good. I
will guarantee someone in qe will find this default and find some
oddball path that tries to allow it to be defined in some read XML file.

Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>

John

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux