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@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to