This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] nl_NL locale: Correct the negative monetary format (bug 24614).
- From: Florian Weimer <fweimer at redhat dot com>
- To: Rafal Luzynski <digitalfreak at lingonborough dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Fri, 07 Jun 2019 09:58:05 +0200
- Subject: Re: [PATCH] nl_NL locale: Correct the negative monetary format (bug 24614).
- References: <1780055980.914645.1559776082012@poczta.nazwa.pl>
* Rafal Luzynski:
> diff --git a/stdlib/tst-strfmon_l.c b/stdlib/tst-strfmon_l.c
> index a193ed6787..7f0f6d47d5 100644
> --- a/stdlib/tst-strfmon_l.c
> +++ b/stdlib/tst-strfmon_l.c
> @@ -216,6 +216,19 @@ static const struct locale_pair tests[] =
> }
> }
> },
> + {
> + "nl_NL.UTF-8",
> + {
> + {
> + "EUR 1\u202f234\u202f567,89", "\u20ac 1\u202f234\u202f567,89",
> + "EUR 1234567,89", "\u20ac 1234567,89"
> + },
> + {
> + "EUR -1\u202f234\u202f567,89", "\u20ac -1\u202f234\u202f567,89",
> + "EUR -1234567,89", "\u20ac -1234567,89",
> + }
> + }
> + },
> {}
> };
Originally, this test was intended as a test for strfmon_l, in
particular that the locale argument controls all aspects of the
formatting, and not just a subset. It is not a general-purpose locale
test.
For locale testing, we should perhaps iterate through all supported
locales and check against test expectations for number and money
formatting. I don't think we do that today.
Thanks,
Florian