Re: [libvirt] [PATCH 4/7] Remove useless variable in qemuDomainAssignAddresses

2016-05-03 Thread Peter Krempa
On Tue, May 03, 2016 at 12:47:24 +0200, Ján Tomko wrote: > We do not need to propagate the exact return values > and the only possible ones are 0 and -1 anyway. > > Remove the temporary variable and use the usual pattern: > > if (f() < 0) > return -1; > --- > src/qemu/qemu_domain_address.c

[libvirt] [PATCH 4/7] Remove useless variable in qemuDomainAssignAddresses

2016-05-03 Thread Ján Tomko
We do not need to propagate the exact return values and the only possible ones are 0 and -1 anyway. Remove the temporary variable and use the usual pattern: if (f() < 0) return -1; --- src/qemu/qemu_domain_address.c | 22 ++ 1 file changed, 10 insertions(+), 12