[libvirt] [PATCH v3 7/7] python: add bindings for virConnectGetCPUModelNames

Giuseppe Scrivano gscrivan at redhat.com
Fri Sep 13 22:59:32 UTC 2013


Eric Blake <eblake at redhat.com> writes:

> On 09/11/2013 08:13 AM, Giuseppe Scrivano wrote:
>> Signed-off-by: Giuseppe Scrivano <gscrivan at redhat.com>
>> ---
>>  python/generator.py             |  2 +-
>>  python/libvirt-override-api.xml |  7 ++++++
>>  python/libvirt-override.c       | 52 +++++++++++++++++++++++++++++++++++++++++
>>  python/libvirt-override.py      | 11 +++++++++
>>  4 files changed, 71 insertions(+), 1 deletion(-)
>> 
>
>> +
>> +    LIBVIRT_BEGIN_ALLOW_THREADS;
>> +
>> +    c_retval = virConnectGetCPUModelNames(conn, arch, &models, flags);
>> +
>> +    LIBVIRT_END_ALLOW_THREADS;
>> +
>> +    if (c_retval == -1)
>> +        return VIR_PY_INT_FAIL;
>> +
>> +    if ((rv = PyList_New(c_retval)) == NULL)
>> +        goto error;
>> +
>> +    for (i = 0; i < c_retval; i++) {
>> +        PyObject *str;
>> +        if ((str = PyString_FromString(models[i])) == NULL)
>> +            goto error;
>> +
>> +        PyList_SET_ITEM(rv, i, str);
>
> Elsewhere, we've used PyList_New(0)/PyList_Append() rather than
> PyList_New(count)/PyList_SET_ITEM(); but that's not universal; also, I
> see uses of PyList_SetItem but not PyList_SET_ITEM; what's the difference?

PyList_SET_ITEM is just an optimized version of PyList_SetItem that
doesn't do any error checking or attempt to release an object already
present in the slot.  It is ok to use here, as we are sure the rv list
is free when we fill it with model names.

I think there are other cases in libvirt-override.c where we can replace
PyList_SetItem with PyList_SET_ITEM without any harm.

Giuseppe




More information about the libvir-list mailing list