This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] nl_NL locale: Correct the negative monetary format (bug 24614).


* 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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]