]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
intl: Fix clang -Wunused-but-set-variable on plural.c
authorAdhemerval Zanella <adhemerval.zanella@linaro.org>
Wed, 24 Aug 2022 14:13:23 +0000 (11:13 -0300)
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>
Tue, 1 Nov 2022 12:45:34 +0000 (09:45 -0300)
Clang warns that '__gettextnerrs' set but not used:

  intl/plural.c:1034:9: error: variable '__gettextnerrs' set but not used
  [-Werror,-Wunused-but-set-variable]
      int yynerrs = 0;
        ^

Clang 15 (https://reviews.llvm.org/D122271) -Wunused-but-set-variable
gives a warning while GCC doesn't.  The -Wunused-but-set-variable is
available in GCC 4.6, lower than the minimum required version 6.2.

Since the file is auto-generated, suppress the warning with a compiler
flag.

Reviewed-by: Fangrui Song <maskray@google.com>
intl/Makefile

index 315c75a18f6a9858046e7c4b05059957d1b73e17..1c3b2a227356a66b026ba01f9414143d2569affb 100644 (file)
@@ -154,7 +154,8 @@ $(objpfx)tst-gettext5.out: $(objpfx)tst-gettext.out
 $(objpfx)tst-gettext6.out: $(objpfx)tst-gettext.out
 
 CPPFLAGS += -D'LOCALEDIR="$(localedir)"' \
-           -D'LOCALE_ALIAS_PATH="$(localedir)"'
+           -D'LOCALE_ALIAS_PATH="$(localedir)"' \
+           -Wno-unused-but-set-variable
 BISONFLAGS = --yacc --no-lines --name-prefix=__gettext --output
 
 $(inst_localedir)/locale.alias: locale.alias $(+force)