On Sun, 2017-06-25 at 15:57 -0400, Laine Stump wrote:
> > @@ -9348,6 +9351,23 @@ virDomainControllerDefParseXML(xmlNodePtr node,
> > goto error;
> > }
> > }
> > + if (idx) {
>
> Oh wait - I thought you were going to rename this to targetIdx (both the
> locals and the member of the struct) to avoid confusion.
>
> So make my ACK conditional on that.
Yeah, that's a good idea. Consider it done.
--
Andrea Bolognani / Red Hat / Virtualization
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list
