On 07/12/2013 06:35 AM, Daniel P. Berrange wrote: > On Thu, Jul 11, 2013 at 07:55:53PM -0400, John Ferlan wrote: >> Add a new qemuMonitorJSONSetObjectProperty() method to support invocation >> of the 'qom-set' JSON monitor command with a provided path, property, and >> expected data type to set. >> >> The test code uses the same "/machine/i440fx" property as the get test and >> attempts to set the "realized" property to "true" (which it should be set >> at anyway). >> --- >> src/qemu/qemu_monitor_json.c | 62 >> ++++++++++++++++++++++++++++++++++++++++++++ >> src/qemu/qemu_monitor_json.h | 6 +++++ >> tests/qemumonitorjsontest.c | 59 +++++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 127 insertions(+) > > ACK
Again, shouldn't this function be marked static, and the .h file left alone? -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
