On Thu, 2016-03-10 at 14:25 -0500, John Ferlan wrote: > <SIGH> Should have read patch 18 & 19 first... Looks like I'm getting > confuzzled by all these lists and multitude of ways names were > generated. The comparison being done is against the copy that came from > the activePCIHostdev list which will have the fields I was concerned > about. So in retrospect... > > ACK To be fair, patches 18 and 19 were written *exactly* to make it a tiny bit easier to follow along, especially with patch 21 in mind. It would probably have been better if the name changes had been made *before* changing the code too much, as this patch is doing, but the comments in patch 19 would not really be true if patch 15 had not been applied first, so... Rock and hard place, really :( I'll reply to your other concern down the thread, consider this just an aside. Cheers. -- Andrea Bolognani Software Engineer - Virtualization Team -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list