Re: [libvirt] [PATCH v3 13/20] cpu: Support multiple PVRs in the ppc64 driver
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