address-type check in mdev post parse

John Ferlan Thu, 28 Jun 2018 13:25:20 -0700


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 <[email protected]>
> ---
>  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 <[email protected]>

John

--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v1 01/11] conf: Remove a redundant model/address-type check in mdev post parse

Reply via email to