From: Junio C Hamano <gitster@pobox.com>
To: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Cc: git@vger.kernel.org, "René Scharfe" <l.s.r@web.de>,
"Jeff King" <peff@peff.net>, "Elijah Newren" <newren@gmail.com>
Subject: Re: [PATCH 0/5] CodingGuidelines: various C99 updates
Date: Fri, 07 Oct 2022 10:54:00 -0700 [thread overview]
Message-ID: <xmqqk05bfrk7.fsf@gitster.g> (raw)
In-Reply-To: <cover-0.5-00000000000-20221007T092505Z-avarab@gmail.com> ("Ævar Arnfjörð Bjarmason"'s message of "Fri, 7 Oct 2022 11:30:29 +0200")
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
> This series:
>
> * Rephrases CodingGuidelines so that we note we're on C99, and then
> lists exceptions and features we use. The previous prose assumed
> C89 by default.
>
> We still explicitly avoid opening the "feel free to use an C99
> feature" floodgates.
The above contradicts with each other. A sensible position to
support the "we do not open the floodgate" is that when in doubt
stick to C89 but use C99 features that are explicitly allowed.
> * Mentions that you can use dynamic C99 initializer elements. See the
> recent discussion at
> https://lore.kernel.org/git/221006.86a668r5mf.gmgdl@evledraar.gmail.com/
Good.
> * Allows us to use "for (int i". I didn't set out to (slightly) jump
> the gun on this, but just pulling the trigger around ~20 days early
> makes it easier to ...
This is a welcome change. As anything this set of patches won't
become reality in any released version until mid December anyway,
this is the cycle to "revisit around November 2022".
> * ...add the natural follow-up section of C99 features you explicitly
> shouldn't be using yet, to which I added the two cases I could
> remember (in 4-5/5).
And we do not have to say we do not use these from C99 if our base
is C89, with explicitly allowed features from C99.
next prev parent reply other threads:[~2022-10-07 17:54 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-07 9:30 [PATCH 0/5] CodingGuidelines: various C99 updates Ævar Arnfjörð Bjarmason
2022-10-07 9:30 ` [PATCH 1/5] CodingGuidelines: update for C99 Ævar Arnfjörð Bjarmason
2022-10-07 18:08 ` Junio C Hamano
2022-10-07 9:30 ` [PATCH 2/5] CodingGuidelines: mention dynamic C99 initializer elements Ævar Arnfjörð Bjarmason
2022-10-07 9:30 ` [PATCH 3/5] CodingGuidelines: allow declaring variables in for loops Ævar Arnfjörð Bjarmason
2022-10-07 18:12 ` Junio C Hamano
2022-10-07 9:30 ` [PATCH 4/5] CodingGuidelines: mention C99 features we can't use Ævar Arnfjörð Bjarmason
2022-10-07 18:13 ` Junio C Hamano
2022-10-07 9:30 ` [PATCH 5/5] CodingGuidelines: recommend against unportable C99 struct syntax Ævar Arnfjörð Bjarmason
2022-10-07 17:54 ` Junio C Hamano [this message]
2022-10-07 18:15 ` [PATCH 0/5] CodingGuidelines: various C99 updates Junio C Hamano
2022-10-10 20:37 ` [PATCH v2 0/5] CodingGUidelines: " Junio C Hamano
2022-10-10 20:37 ` [PATCH v2 1/5] CodingGuidelines: update for C99 Junio C Hamano
2022-10-10 20:37 ` [PATCH v2 2/5] CodingGuidelines: mention dynamic C99 initializer elements Junio C Hamano
2022-10-10 20:37 ` [PATCH v2 3/5] CodingGuidelines: allow declaring variables in for loops Junio C Hamano
2022-10-10 20:37 ` [PATCH v2 4/5] CodingGuidelines: mention C99 features we can't use Junio C Hamano
2022-10-10 20:38 ` [PATCH v2 5/5] CodingGuidelines: recommend against unportable C99 struct syntax Junio C Hamano
2022-10-11 0:09 ` Jeff King
2022-10-11 0:26 ` Junio C Hamano
2022-10-11 0:39 ` Jeff King
2022-10-11 8:30 ` Ævar Arnfjörð Bjarmason
2022-10-11 15:01 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqk05bfrk7.fsf@gitster.g \
--to=gitster@pobox.com \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
--cc=l.s.r@web.de \
--cc=newren@gmail.com \
--cc=peff@peff.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).