Re: [libvirt] [PATCH 2/3] qemu_monitor_json: Drop redundant checks

2016-05-03 Thread Peter Krempa
On Tue, May 03, 2016 at 11:53:20 +0200, Michal Privoznik wrote: > In these functions I'm fixing here, we do call > qemuMonitorJSONCheckError() followed by another check if qemu > reply contains 'return' object. If it wouldn't, the former > CheckError() function would error out and the flow would

[libvirt] [PATCH 2/3] qemu_monitor_json: Drop redundant checks

2016-05-03 Thread Michal Privoznik
In these functions I'm fixing here, we do call qemuMonitorJSONCheckError() followed by another check if qemu reply contains 'return' object. If it wouldn't, the former CheckError() function would error out and the flow would not even get to the latter. Signed-off-by: Michal Privoznik