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

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

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...


Xen-devel mailing list



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