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

Re: [Xen-devel] [PATCH v3] libxl: Spice vdagent support for upstream qemu

On Tue, May 07, 2013 at 11:18:43AM +0100, Ian Campbell wrote:
> > 
> > What I meant is in the documentation we should use true / false instead
> > of 1 / 0. Other options use true / false as well, like e820_host and
> > spiceagent_mouse.
> Or right, we do seem to be rather inconsistent about that.
> Given that the parser (currently) only understands 0 and 1 (or other
> integers I guess) I think the most consistent usage would be to write
> "true (1)" or "false (0)" everywhere.
> Do you fancy putting together a patch to make this so?

We already have clear definition of BOOLEAN type around line 37 in the same
file. IMHO using "true" / "false" is sufficent.


> Ian.

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.