diff options
author | Mo Yuxiang <Moyuxiang@huawei.com> | 2014-08-08 17:55:37 +0800 |
---|---|---|
committer | Ján Tomko <jtomko@redhat.com> | 2014-08-19 12:43:28 +0200 |
commit | 2175f4f3da4dbf5f44a00275bd61f807cedf72ea (patch) | |
tree | a811db905a5142b53cc03df6820fb295756c7c34 /libvirt-qemu-override.c | |
parent | 86ee51ce97525fd474d3a155d67bf0730597c23a (diff) | |
download | libvirt-python-2175f4f3da4dbf5f44a00275bd61f807cedf72ea.tar.gz |
build: Fix build warning on libvirt-python
On compiling libvirt-python, we get such a warning:
libvirt-qemu-override.c: In function ‘libvirt_qemu_virConnectDomainQemuMonitorEventRegister’:
libvirt-qemu-override.c:304: warning: suggest explicit braces to avoid ambiguous ‘else’
Py_DECREF is a macro using if/else on older Python releases.
The solution is to add braces.
Python 2.7 and newer has the macro wrapped in a do { } while(0) block.
Signed-off-by: Mo Yuxiang <Moyuxiang@huawei.com>
Signed-off-by: Ján Tomko <jtomko@redhat.com>
Diffstat (limited to 'libvirt-qemu-override.c')
-rw-r--r-- | libvirt-qemu-override.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libvirt-qemu-override.c b/libvirt-qemu-override.c index 05ead30..8be3755 100644 --- a/libvirt-qemu-override.c +++ b/libvirt-qemu-override.c @@ -301,8 +301,9 @@ libvirt_qemu_virConnectDomainQemuMonitorEventRegister(PyObject *self ATTRIBUTE_U flags); LIBVIRT_END_ALLOW_THREADS; - if (ret < 0) + if (ret < 0) { Py_DECREF(pyobj_cbData); + } py_retval = libvirt_intWrap(ret); return py_retval; |