On Fri, Oct 30, 2009 at 04:28:49PM +0000, Jamie Lokier wrote: > Anthony Liguori wrote: > > Okay, let's get more clever then and do: > > > > #define QERR_DEV_NFOUND "{ 'code': 404, 'name': %s}" > > By the way, since you've already invented a non-standard JSON > extension, which is the single quotes, why not go a step further and > permit the quotes to be omitted for simple tokens?
If we're going to use JSON we should be 100% compliant with the JSON spec, not extend it. By adding custom QEMU extensions, we loose the ability for programming language to trivially talk to QEMU using a standard JSON parser, and instead everyone has to write custom client side code yet again. Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|