> From: Eric Blake [mailto:[email protected]] > Subject: Re: [Qemu-devel] [PATCH] qmp: using genneric string paring function > for 'qom-set' > > On 08/27/2014 11:19 PM, [email protected] wrote: > > From: Gonglei <[email protected]> > > > > In the subject: s/genneric/generic/, s/paring/parsing/ > OK.
> > At present, 'qom-set' only can set string type property, > > qom-set is typed as taking arbitrary JSON 9one of the few commands with > 'gen':'no', which means it is completely type-unsafe, and can therefore > accept EVERY type, including JSON integers, not just strings). > Yep. Thanks, Eric. This is my fault. > > which will restrict this qmp command's function. > > Using genneric string paring function can handle different > > Same typos here as in subject. > Yep. Thanks for your review. Best regards, -Gonglei > > types, such as int/bool/string etc. > > > > Signed-off-by: Gonglei <[email protected]> > > --- > > -- > Eric Blake eblake redhat com +1-919-301-3266 > Libvirt virtualization library http://libvirt.org
