Re: [PATCH v3 03/10] Add qemuMonitorJSONSetObjectProperty() method for QMP qom-set command

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/12/2013 11:57 AM, Eric Blake wrote:
> 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?
> 

I can go either way on this - without the .h changes, we don't have the
test though.  So whichever way is preferred - I'll go with it...

John

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]