Re: [PATCH xf86-video-qxl 4/4] Document a number of missing XSpice vdagent options in the example xorg.conf.

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

 



On Fri, Oct 16, 2015 at 04:07:43PM -0500, Jeremy White wrote:
> On 10/16/2015 11:04 AM, Christophe Fergeau wrote:
> > On Fri, Oct 02, 2015 at 11:21:03AM -0500, Jeremy White wrote:
> >> Signed-off-by: Jeremy White <jwhite@xxxxxxxxxxxxxxx> --- 
> >> examples/spiceqxl.xorg.conf.example | 24
> >> ++++++++++++++++++++++++ 1 file changed, 24 insertions(+)
> >> 
> >> diff --git a/examples/spiceqxl.xorg.conf.example
> >> b/examples/spiceqxl.xorg.conf.example index 9b223c8..e709e6d
> >> 100644 --- a/examples/spiceqxl.xorg.conf.example +++
> >> b/examples/spiceqxl.xorg.conf.example @@ -126,6 +126,30 @@
> >> Section "Device" # defaults to 16 #Option "FrameBufferSize" "16"
> >> 
> >> +    # Whether or not to accept Vdagent connections +    #
> >> defaults to false
> > 
> > Nit: should be
> >> +    #  default: False
> > 
> > in order to be consisent with the others.
> 
> Well, actually, that file doesn't really have a consistent pattern for
> expressing the default.

Yeah, I saw the whole file afterwards, the consistency I was thinking of
was only within the patch, "default: " is used everywhere in that patch
except for this one.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]