This is the mail archive of the libc-alpha@sources.redhat.com 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] Correct formatting of internatinal currency (third try)


[Ulrich Drepper]
> I've applied the patches now, but with some changes.  strfmon cannot use
> a static variable.  The code must be thread safe.

Ah, good point.

> There is also no need to create yet more locales.  Instead the int_*
> locale content should simply go in the already existing files.

Actually, there is a reason to create new test locales.  One want to
detect when the domestic values are used instead of the international
values.  I did this by making sure the int_ locales used the default
(ie C/POSIX locale) domestic values, and only changed the
international values.  For this to work, strfmon() have to use the
international values.  The error in strfmon() would not have been
detected with the test files in the current CVS.  It would have been
detected with my suggested test set.

> The seems OK.  Thanks,

Thank you too for your good job as the glibc gatekeeper. :)


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