Re: [Qemu-devel] Re: [PATCH 10/11] Add a QObject JSON wrapper

2009-11-11 Thread Luiz Capitulino
On Wed, 11 Nov 2009 13:39:29 -0600 Anthony Liguori wrote: > Luiz Capitulino wrote: > > I think we should abort() on error, assuming the only way to fail > > is a bad syntax. > > > > We'll be using qobject_from_jsonf() to parse incoming QMP traffic. We > definitely don't want to abort when

Re: [Qemu-devel] Re: [PATCH 10/11] Add a QObject JSON wrapper

2009-11-11 Thread Anthony Liguori
Luiz Capitulino wrote: I think we should abort() on error, assuming the only way to fail is a bad syntax. We'll be using qobject_from_jsonf() to parse incoming QMP traffic. We definitely don't want to abort when we receive invalid input on the QMP port. Best to just close the session

[Qemu-devel] Re: [PATCH 10/11] Add a QObject JSON wrapper

2009-11-11 Thread Luiz Capitulino
On Wed, 11 Nov 2009 11:29:02 -0600 Anthony Liguori wrote: > This provides a QObject interface for creating QObjects from a JSON > expression. > > Signed-off-by: Anthony Liguori > --- > Makefile |2 +- > qjson.c | 60 > qjson.