Re: [libvirt PATCH 0/8] Remove space-padded alignment from public headers
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(a)redhat.com>
(I'll fix the nit if you're done with this series)
Regards,
Erik