Re: [libvirt] [PATCH v3 13/20] cpu: Support multiple PVRs in the ppc64 driver

11 Aug 2015

      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