On Mon, Jun 03, 2013 at 18:32:15 +0800, Osier Yang wrote: > On 03/06/13 18:22, Jiri Denemark wrote: > > Using an uninitialized value and a bool saying if the value is valid may > > confuse compilators. ... > This statement still uses the bool variable "found": > > hostdev->info->addr.drive.controller = found ? > def->controllers[i - 1]->idx : > nscsi_controllers; > Heh, I should have looked at the compilation result before sending this patch :-) V2 is on the way. Jirka -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list