Re: [libvirt] [PATCH v1 01/11] conf: Remove a redundant model/address-type check in mdev post parse
On 06/27/2018 09:34 AM, Erik Skultety wrote:
It's pointless to check the same thing multiple times.
Fix the indentation along the way too.
Signed-off-by: Erik Skultety <eskultet(a)redhat.com>
---
src/conf/domain_conf.c | 14 ++------------
1 file changed, 2 insertions(+), 12 deletions(-)
I think the reason for the separate checks was fear of messing with the
current check when ccw addressing was added ;-)
Reviewed-by: John Ferlan <jferlan(a)redhat.com>
John