Re: [Xen-devel] [PATCH 8/9] xl: xl block-attach -N (dry run) option

[Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>]

On Fri, 2011-06-03 at 12:16 +0100, Ian Jackson wrote:
[...]
> Just the entire structure.
[...]
> JSON would be a good concrete syntax.
[...]
> If we had a competent JSON pretty-printer it would do as the
> human-readable output too.

Should be pretty easy. QMP support in libxl is also going to require a
JSON library (for internal use by libxl, not exposed to libxl users),
which hopefully has functions which make the pretty part trivial.

>   We shouldn't be encouraging XML :-).

:-D

> > What do you think? (I think I lean towards the final option, unless a
> > single hardcoded syntax is thought to be sufficient).
> 
> The prototype you suggest is not sufficient because field members may
> be structures, so the whole thing needs to be recursive.  If you want
> reasonable output from a recursive pretty-printer you need to pass an
> indent level etc.

Oh yeah. Add more params to callback as necessary ;-)

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel