RE: [libvirt] [PATCH] add fullscreen support to qemu sdl xml (via fullscreen='true' attribute for the graphics element)

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

 



> -----Original Message-----
> From: Daniel P. Berrange [mailto:berrange@xxxxxxxxxx]
> In this bit of the code:
> 
> diff --git a/src/domain_conf.h b/src/domain_conf.h
> index 51cf6d5..436d0b1 100644
> --- a/src/domain_conf.h
> +++ b/src/domain_conf.h
> @@ -268,6 +268,7 @@ struct _virDomainGraphicsDef {
>          struct {
>              char *display;
>              char *xauth;
> +            char *fullscreen;
>          } sdl;
>      } data;
>  };
> 
> 
> 
> You shouldn't use a char * for fullscreen. This is a boolean flag, so
> it should just be an bitfield  'int fullscreen: 1'. This avoids the
> issue DV pointed out of unexpected values in the XML.
[IH] this is used to get the string from the xml, to compare it to the
string 'true'.
Will the xml parsing infrastructure automatically convert true/false to
int and handle parsing errors or I need to write that code in another
place?

--
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]