On Wed, 2019-02-13 at 19:03 +0000, Daniel P. Berrangé wrote:
[...]
> + pkgname = temp[item]
> + if pkgname is None:
> + continue
> + if pkgname not in pkgs:
> + pkgs.append(pkgname)
I think this would look slightly better as
if pkgname is None:
continue
if pkgname in pkgs:
continue
pkgs.append(pkgname)
Regardless of whether you change it or keep it as-is,
Reviewed-by: Andrea Bolognani <[email protected]>
--
Andrea Bolognani / Red Hat / Virtualization
--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list
