Re: [libvirt] [PATCH v2 3/4] qemu: Move vm->persistent check into qemuDomainRemoveInactive

2015-09-24 Thread Martin Kletzander
On Wed, Sep 23, 2015 at 03:00:36PM +0200, Michal Privoznik wrote: So far we have the following pattern occurring over and over again: if (!vm->persistent) qemuDomainRemoveInactive(driver, vm); You could've done that earlier so you would save some lines in the patches before. Anyway, ni

[libvirt] [PATCH v2 3/4] qemu: Move vm->persistent check into qemuDomainRemoveInactive

2015-09-23 Thread Michal Privoznik
So far we have the following pattern occurring over and over again: if (!vm->persistent) qemuDomainRemoveInactive(driver, vm); It's safe to put the check into the function and save some LoC. Signed-off-by: Michal Privoznik --- src/qemu/qemu_domain.c| 9 - src/qemu/qemu_dri