linux-kernel - Re: [PATCH] scripts/unifdef: avoid constexpr keyword

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <196e71e0-6b17-f970-c034-4452e0c44052@dotat.at>
Date: Thu, 21 Mar 2024 15:53:59 +0000
From: Tony Finch <dot@...at.at>
To: Arnd Bergmann <arnd@...nel.org>
cc: Sam Ravnborg <sam@...nborg.org>, Arnd Bergmann <arnd@...db.de>, 
    Masahiro Yamada <masahiroy@...nel.org>, 
    Nathan Chancellor <nathan@...nel.org>, Nicolas Schier <nicolas@...sle.eu>, 
    linux-kbuild@...r.kernel.org, Michal Marek <mmarek@...e.cz>, 
    linux-kernel@...r.kernel.org
Subject: Re: [PATCH] scripts/unifdef: avoid constexpr keyword

Arnd Bergmann <arnd@...nel.org> wrote:
>
> Starting with c23, 'constexpr' is a keyword in C like in C++ and cannot
> be used as an identifier:

So it is! Can you please incorporate the fixup patch below? Other than
that, LGTM.

Reviewed-By: Tony Finch <dot@...at.at>

PS. I've been reviewing some other details of C23 recently. According to
the Editor's Report https://open-std.org/jtc1/sc22/wg14/www/docs/n3221.htm
the working draft that is closest to the official C23 Draft International
Standard is https://open-std.org/jtc1/sc22/wg14/www/docs/n3220.pdf
(There were significant changes and fixes even quite late in the process
so don't rely on earlier versions - I encoutered a bug when I did!)


--- scripts/unifdef.c
+++ scripts/unifdef.c
@@ -819,7 +819,7 @@ static const struct ops {
 /*
  * Function for evaluating the innermost parts of expressions,
  * viz. !expr (expr) number defined(symbol) symbol
- * We reset the constexpr flag in the last two cases.
+ * We reset the constexpression flag in the last two cases.
  */
 static Linetype
 eval_unary(const struct ops *ops, int *valp, const char **cpp)


-- 
Tony Finch  <dot@...at.at>  https://dotat.at/
South Utsire: Variable 2 to 4 becoming south or southwest 5 to 7.
Moderate or rough. Rain or showers. Moderate or good, occasionally
poor.