ToT glibc build problem with ToT GCC
Florian Weimer
fweimer@redhat.com
Fri Aug 30 13:06:00 GMT 2019
* Carlos O'Donell:
> Is there a viable solution for a static allocation of a structure
> that ends in a VLA?
Sure, use an initializer. Not sure if this breaks anything. The test
suite is clean.
(GNU C obviously didn't support that when the code was written.)
Thanks,
Florian
localedef: Use initializer for flexible array member [BZ #24950]
2019-08-30 Florian Weimer <fweimer@redhat.com>
[BZ #24950]
* locale/programs/charmap.h (struct charseq): Turn bytes into a
flexible array member.
* locale/programs/ld-ctype.c (ctype_finish): Use initializer for
replace.
diff --git a/locale/programs/charmap.h b/locale/programs/charmap.h
index 870a9e9577..70db330d29 100644
--- a/locale/programs/charmap.h
+++ b/locale/programs/charmap.h
@@ -60,7 +60,7 @@ struct charseq
const char *name;
uint32_t ucs4;
int nbytes;
- unsigned char bytes[0];
+ unsigned char bytes[];
};
diff --git a/locale/programs/ld-ctype.c b/locale/programs/ld-ctype.c
index cfc9c43fd5..12d05f1641 100644
--- a/locale/programs/ld-ctype.c
+++ b/locale/programs/ld-ctype.c
@@ -842,8 +842,6 @@ no input digits defined and none of the standard names in the charmap"));
for (cnt = 0; cnt < 10; ++cnt)
if (ctype->mboutdigits[cnt] == NULL)
{
- static struct charseq replace[2];
-
if (!warned)
{
record_error (0, 0, _("\
@@ -851,10 +849,12 @@ not all characters used in `outdigit' are available in the charmap"));
warned = 1;
}
- replace[0].nbytes = 1;
- replace[0].bytes[0] = '?';
- replace[0].bytes[1] = '\0';
- ctype->mboutdigits[cnt] = &replace[0];
+ static const struct charseq replace =
+ {
+ .nbytes = 1,
+ .bytes = { '\0' },
+ };
+ ctype->mboutdigits[cnt] = (struct charseq *) &replace;
}
warned = 0;
More information about the Libc-alpha
mailing list