On Tue, 2015-08-11 at 10:26 +0200, Jiri Denemark wrote: > > > + if (!virXPathBoolean("boolean(./@value)", ctxt) || > > + virXPathULongHex("string(./@value)", ctxt, &pvr) < 0) > > { > > + virReportError(VIR_ERR_INTERNAL_ERROR, > > + _("Missing or invalid PVR value in > > CPU model %s"), > > + model->name); > > + goto ignore; > > Wrong indentation, s/^ // in the 4 lines above. ACK once it's > fixed. Good catch. I've fixed the indentation and removed the unnecessary call to virXPathBoolean(), as agreed, both here and in patch 15/20. Cheers. -- Andrea Bolognani Software Engineer - Virtualization Team -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list