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

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

 



On Thu, 21 Jan 2021 08:38:06 +0100
Erik Skultety <eskultet@xxxxxxxxxx> 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@xxxxxxxxxx>
> (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.




[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]

  Powered by Linux