[Spice-devel] [PATCH xf86-video-qxl] Xspice: handle parameters with value 0, allows --port 0

Christophe Fergeau cfergeau at redhat.com
Tue Nov 3 08:33:11 PST 2015


Why do you want to allow that? (can you add this to the commit log?)

Christophe

On Mon, Nov 02, 2015 at 04:14:16PM -0600, Jeremy White wrote:
> Signed-off-by: Jeremy White <jwhite at codeweavers.com>
> ---
>  scripts/Xspice | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/Xspice b/scripts/Xspice
> index 52b1b16..46c5c1e 100755
> --- a/scripts/Xspice
> +++ b/scripts/Xspice
> @@ -256,7 +256,7 @@ var_args = ['port', 'tls_port', 'disable_ticketing',
>      'vdagent_uid', 'vdagent_gid']
>  
>  for arg in var_args:
> -    if getattr(args, arg):
> +    if getattr(args, arg) != None:
>          # The Qxl code doesn't respect booleans, so pass them as 0/1
>          a = getattr(args, arg)
>          if a == True:
> -- 
> 2.1.4
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20151103/d55de69a/attachment.sig>


More information about the Spice-devel mailing list