[libvirt] [PATCH] python: make other APIs share common {get, set}PyVirTypedParameter
Eric Blake
eblake at redhat.com
Sat Feb 11 00:18:44 UTC 2012
On 02/10/2012 03:17 AM, Guannan Ren wrote:
> *libvirt_virDomainBlockStatsFlags
> *libvirt_virDomainGetSchedulerParameters
> *libvirt_virDomainGetSchedulerParametersFlags
> *libvirt_virDomainSetSchedulerParameters
> *libvirt_virDomainSetSchedulerParametersFlags
> *libvirt_virDomainSetBlkioParameters
> *libvirt_virDomainGetBlkioParameters
> *libvirt_virDomainSetMemoryParameters
> *libvirt_virDomainGetMemoryParameters
> *libvirt_virDomainSetBlockIoTune
> *libvirt_virDomainGetBlockIoTune
> ---
> python/libvirt-override-api.xml | 12 +-
> python/libvirt-override.c | 885 +++++++++++++--------------------------
> 2 files changed, 295 insertions(+), 602 deletions(-)
Looks like your patch overlaps with Alex's patch.
> static PyObject *
> libvirt_virDomainSetSchedulerParameters(PyObject *self ATTRIBUTE_UNUSED,
> - PyObject *args) {
> + PyObject *args)
> +{
> virDomainPtr domain;
> PyObject *pyobj_domain, *info;
> + PyObject *ret = NULL;
> char *c_retval;
> int i_retval;
> - int nparams, i;
> - virTypedParameterPtr params;
> + int nparams = 0, size = 0;
size should be Py_ssize_t, throughout the patch. Other than that, looks
pretty mechanical, and I didn't spot anything obviously wrong.
ACK and pushed.
--
Eric Blake eblake at 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: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120210/0be44c11/attachment-0001.sig>
More information about the libvir-list
mailing list