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

Erik Skultety eskultet at redhat.com
Thu Jan 21 07:38:06 UTC 2021


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




More information about the libvir-list mailing list