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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]