[libvirt] [PATCH v2 09/12] qemuDomainDetachDeviceLiveAndConfig: Avoid overwriting @ret

Ján Tomko jtomko at redhat.com
Sat May 26 08:40:05 UTC 2018


On Thu, May 24, 2018 at 01:13:36PM +0200, Michal Privoznik wrote:
>The fact that we are overwriting @ret multiple times is very
>confusing to see what is actually happening here.

-EPARSE
s/is very confusing/makes it difficult to see/

>Follow our
>traditional pattern where @ret is initialized to -1, and set to 0
>only in case we know we succeeded.
>
>Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>---
> src/qemu/qemu_driver.c | 24 ++++++++++++------------
> 1 file changed, 12 insertions(+), 12 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko at redhat.com>

Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180526/c56d1256/attachment-0001.sig>


More information about the libvir-list mailing list