Re: [Xen-devel] [PATCH] allow xendomains to work for xl list -l

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Fri, 2013-04-12 at 00:02 +0100, M A Young wrote:
> On Thu, 11 Apr 2013, Ian Campbell wrote:
> 
> > On Thu, 2013-04-11 at 00:15 +0100, M A Young wrote:
> >> Yes, setting yajl_gen_beautify (as in the attached patch) gets the
> >> xendomains script working again. I didn't try setting the spaces though
> >> that does indeed seem to be the default.
> >
> > Cool. I think you need to check for errors after
> >     g = yajl_gen_alloc(allocFuncs);
> > though.
> 
> This version of the patch checks g isn't NULL before trying to set 
> yajl_gen_beautify . I decided not to test whether yajl_gen_beautify is set 
> successfully since unbeautified output is probably better than nothing.

Agreed. Acked + applied.

Your attachment had DOS line endings for some reason...

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel