[libvirt] [PATCH] storage: Remove redundant refreshPool check

Martin Kletzander mkletzan at redhat.com
Thu Jun 23 07:32:30 UTC 2016


On Wed, Jun 22, 2016 at 08:29:35PM -0400, Cole Robinson wrote:
>Every driver provides a refreshPool impl, and many other critical
>places in the code unconditionally call it without checking if
>it exists, so this check is pointless

I'm not entirely sure about it, but it'd be nicer if we actually checked
that it's non-NULL.  Just to future-proof the code in case someone adds
another backend.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160623/5a965a3c/attachment-0001.sig>


More information about the libvir-list mailing list