[libvirt] [PATCH v3 03/10] Add qemuMonitorJSONSetObjectProperty() method for QMP qom-set command
Eric Blake
eblake at redhat.com
Fri Jul 12 15:57:28 UTC 2013
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130712/ea6627cf/attachment-0001.sig>
More information about the libvir-list
mailing list