[libvirt] [PATCH v2 1/3] qemu: Make all SPICE command-line args optional

[Christophe Fergeau]

On Fri, Mar 18, 2016 at 10:25:58AM +0100, Ján Tomko wrote:
> > -    if (graphics->data.spice.playback)
> > -        virBufferAsprintf(&opt, ",playback-compression=%s",
> > +    }
> > +    if (graphics->data.spice.playback) {
> > +        virBufferAsprintf(&opt, "playback-compression=%s,",
> >                            virTristateSwitchTypeToString(graphics->data.spice.playback));
> > -    if (graphics->data.spice.streaming)
> > -        virBufferAsprintf(&opt, ",streaming-video=%s",
> > +    }
> > +    if (graphics->data.spice.streaming) {
> > +        virBufferAsprintf(&opt, "streaming-video=%s,",
> >                            virDomainGraphicsSpiceStreamingModeTypeToString(graphics->data.spice.streaming));
> > -    if (graphics->data.spice.copypaste == VIR_TRISTATE_BOOL_NO)
> > -        virBufferAddLit(&opt, ",disable-copy-paste");
> > +    }
> > +    if (graphics->data.spice.copypaste == VIR_TRISTATE_BOOL_NO) {
> > +        virBufferAddLit(&opt, "disable-copy-paste,");
> > +    }
> 
> This breaks make syntax-check:
> 
> Curly brackets around single-line body:
> src/qemu/qemu_command.c:7559-7561:
>     if (graphics->data.spice.copypaste == VIR_TRISTATE_BOOL_NO) {
>         virBufferAddLit(&opt, "disable-copy-paste,");
>     }

Ah thanks, I'll make sure to fix these before pushing. Totally forgot to
rerun it before sending v2.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160318/5515c40a/attachment-0001.sig>