ToT glibc build problem with ToT GCC
Commit Message
* 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.
Comments
On 30/08/2019 14:06, Florian Weimer wrote:
> * 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.)
i didn't know an initializer would be enough.
>
> - 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;
> }
is it ok to change {'?','\0'} to {'\0'} ?
@@ -60,7 +60,7 @@ struct charseq
const char *name;
uint32_t ucs4;
int nbytes;
- unsigned char bytes[0];
+ unsigned char bytes[];
};
@@ -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;