On Fri, 2019-02-15 at 14:25 +0100, Ján Tomko wrote: > On Fri, Feb 15, 2019 at 12:55:48PM +0100, Andrea Bolognani wrote: > > @@ -4855,6 +4855,51 @@ virDomainRNGDefPostParse(virDomainRNGDefPtr rng) > > return 0; > > } > > > > + > > ^ Whitespace Watch would worry. That's because I added one empty line between functions instead of two in the previous commit. Good catch! > > +static int > > +virDomainDiskDefPostParse(virDomainDiskDefPtr disk, > > + const virDomainDef *def, > > + virDomainXMLOptionPtr xmlopt) > > Unintended indentation ---^ *unindented -- Andrea Bolognani / Red Hat / Virtualization