Can we move Samba to C99 and past -Werror=declaration-after-statement ?

Martin Schwenke martin at meltin.net
Thu Nov 9 22:13:33 UTC 2023


On Fri, 10 Nov 2023 10:58:04 +1300, Andrew Bartlett via samba-technical
<samba-technical at lists.samba.org> wrote:

> Samba chooses to, for it's C style guide, to enforce -
> Werror=declaration-after-statement
> 
> I understand that some prefer the style.
> 
> However C has moved on as a language, and libraries we use (Python 3.12
> in this case) don't compile with it, and we are having to do quite some
> contortions in 
> https://gitlab.com/samba-team/samba/-/merge_requests/3373 to build on
> Fedora 39.
> 
> See also discussion at https://bugzilla.samba.org/show_bug.cgi?id=15513
> 
> Can we agree to just remove this requirement?  
> 
> It will allow us to keep variables closer to their use, which is
> helpful in a lot of ways, particularly in longer functions. 

I think that sounds good.  Yes from me.

Can we also please declare for loop variable in the loop?

peace & happiness,
martin



More information about the samba-technical mailing list