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 <jwh...@codeweavers.com> --- 
> >> 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@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to