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