[libvirt PATCH 0/8] Remove space-padded alignment from public headers

Jonathon Jongsma jjongsma at redhat.com
Thu Jan 21 15:49:37 UTC 2021


On Thu, 21 Jan 2021 08:38:06 +0100
Erik Skultety <eskultet at redhat.com> wrote:

> On Wed, Jan 20, 2021 at 11:33:41AM -0600, Jonathon Jongsma wrote:
> > Erik Skultety suggested posting a separate series removing the
> > space-padded alignment of function declarations in the public
> > headers.  
> 
> This is already an improvement. We've gradually adopted a style where
> function declarations are separated by a single empty line and
> definitions by 2 (although, as with anything we're not 100%
> consistent all the time). With that said, we haven't converted the
> headers to follow this style yet (I remember John Ferlan did this
> conversion for most of the sources in the past).
> 
> Now, it depends whether you want to waste a few more minutes on
> fixing the style in headers or not, I'm okay both ways - if you think
> it's a waste of your time, just let me know and we'll create another
> warmup bitesized task for libvirt newcomers.
> 
> Regardless of how you decide about the code style:
> 
> Reviewed-by: Erik Skultety <eskultet at redhat.com>
> (I'll fix the nit if you're done with this series)
> 
> Regards,
> Erik

It shouldn't take much time. I'll try to go through and revise it.




More information about the libvir-list mailing list