* stdlib/strfmon.c: Move somewhat closer to the standard wrt to
	handling the internation currency symbol.  POSIX says that the
	fourth character is used to separate the currency symbol from the
	value.  Therefore it does not have to be printed.  But we cannot
	remove the space if the currency symbol is printed before the
	number since this is what many locales expect.
	* localedata/tests-mbwc/dat_strfmon.c: Remove #ifdefs introduced to
	work around DEM problem.
	* localedata/tst-fmon.data: Change back entries with DEM.
This commit is contained in:
Ulrich Drepper 2000-07-01 00:39:39 +00:00
parent 7f4553513c
commit 3da67f9970
4 changed files with 39 additions and 69 deletions

View File

@ -1,5 +1,15 @@
2000-06-30 Ulrich Drepper <drepper@redhat.com> 2000-06-30 Ulrich Drepper <drepper@redhat.com>
* stdlib/strfmon.c: Move somewhat closer to the standard wrt to
handling the internation currency symbol. POSIX says that the
fourth character is used to separate the currency symbol from the
value. Therefore it does not have to be printed. But we cannot
remove the space if the currency symbol is printed before the
number since this is what many locales expect.
* localedata/tests-mbwc/dat_strfmon.c: Remove #ifdefs introduced to
work around DEM problem.
* localedata/tst-fmon.data: Change back entries with DEM.
* iconv/gconv_trans.c: Test with _NL_CTYPE_TRANSLIT_DEFAULT_MISSING_LEN * iconv/gconv_trans.c: Test with _NL_CTYPE_TRANSLIT_DEFAULT_MISSING_LEN
whether this information is available. whether this information is available.

View File

@ -20,23 +20,13 @@ TST_STRFMON tst_strfmon_loc [] = {
{ {
{ {
/* #01 */ /* #01 */
#ifdef NO_WAIVER
/*inp*/ { 23, "%n %% %i", 123.00 }, /*inp*/ { 23, "%n %% %i", 123.00 },
/*exp*/ { 1,0,1,22, "123,00 DM % 123,00 DEM" }, /*exp*/ { 1,0,1,22, "123,00 DM % 123,00 DEM" },
#else
/*inp*/ { 24, "%n %% %i", 123.00 },
/*exp*/ { 1,0,1,23, "123,00 DM % 123,00 DEM " },
#endif
}, },
{ {
/* #02 */ /* #02 */
#ifdef NO_WAIVER
/*inp*/ { 23, "%n %% %i", 123.00 }, /*inp*/ { 23, "%n %% %i", 123.00 },
/*exp*/ { 1,0,1,22, "123,00 DM % 123,00 DEM" }, /*exp*/ { 1,0,1,22, "123,00 DM % 123,00 DEM" },
#else
/*inp*/ { 24, "%n %% %i", 123.00 },
/*exp*/ { 1,0,1,23, "123,00 DM % 123,00 DEM " },
#endif
}, },
{ {
/* #03 */ /* #03 */
@ -46,101 +36,57 @@ TST_STRFMON tst_strfmon_loc [] = {
{ {
/* #04 */ /* #04 */
/*inp*/ { 30, "%n|%i", 1234.561 }, /*inp*/ { 30, "%n|%i", 1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,24, "1.234,56 DM|1.234,56 DEM" }, /*exp*/ { 1,0,1,24, "1.234,56 DM|1.234,56 DEM" },
#else
/*exp*/ { 1,0,1,25, "1.234,56 DM|1.234,56 DEM " },
#endif
}, },
{ {
/* #05 */ /* #05 */
/*inp*/ { 32, "%n|%i", -1234.561 }, /*inp*/ { 32, "%n|%i", -1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,26, "-1.234,56 DM|-1.234,56 DEM"}, /*exp*/ { 1,0,1,26, "-1.234,56 DM|-1.234,56 DEM"},
#else
/*exp*/ { 1,0,1,27, "-1.234,56 DM|-1.234,56 DEM "},
#endif
}, },
{ {
/* #06 */ /* #06 */
/*inp*/ { 32, "%12n|%12i", 1234.561 }, /*inp*/ { 32, "%12n|%12i", 1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,25, " 1.234,56 DM|1.234,56 DEM"}, /*exp*/ { 1,0,1,25, " 1.234,56 DM|1.234,56 DEM"},
#else
/*exp*/ { 1,0,1,26, " 1.234,56 DM|1.234,56 DEM "},
#endif
}, },
{ {
/* #07 */ /* #07 */
/*inp*/ { 32, "%12n|%12i", -1234.561 }, /*inp*/ { 32, "%12n|%12i", -1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,26, "-1.234,56 DM|-1.234,56 DEM"}, /*exp*/ { 1,0,1,26, "-1.234,56 DM|-1.234,56 DEM"},
#else
/*exp*/ { 1,0,1,27, "-1.234,56 DM|-1.234,56 DEM "},
#endif
}, },
{ {
/* #08 */ /* #08 */
/*inp*/ { 32, "%#5n|%#5i", 1234.561 }, /*inp*/ { 32, "%#5n|%#5i", 1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,28, " 1.234,56 DM| 1.234,56 DEM"}, /*exp*/ { 1,0,1,28, " 1.234,56 DM| 1.234,56 DEM"},
#else
/*exp*/ { 1,0,1,29, " 1.234,56 DM| 1.234,56 DEM "},
#endif
}, },
{ {
/* #09 */ /* #09 */
/*inp*/ { 32, "%#5n|%#5i", -1234.561 }, /*inp*/ { 32, "%#5n|%#5i", -1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,28, "- 1.234,56 DM|- 1.234,56 DEM"}, /*exp*/ { 1,0,1,28, "- 1.234,56 DM|- 1.234,56 DEM"},
#else
/*exp*/ { 1,0,1,29, "- 1.234,56 DM|- 1.234,56 DEM "},
#endif
}, },
{ {
/* #10 */ /* #10 */
/*inp*/ { 32, "%=*#5n|%=*#5i", 1234.561 }, /*inp*/ { 32, "%=*#5n|%=*#5i", 1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,28, " *1.234,56 DM| *1.234,56 DEM"}, /*exp*/ { 1,0,1,28, " *1.234,56 DM| *1.234,56 DEM"},
#else
/*exp*/ { 1,0,1,29, " *1.234,56 DM| *1.234,56 DEM "},
#endif
}, },
{ {
/* #11 */ /* #11 */
/*inp*/ { 32, "%=0#5n|%=0#5i", -1234.561 }, /*inp*/ { 32, "%=0#5n|%=0#5i", -1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,28, "-01.234,56 DM|-01.234,56 DEM"}, /*exp*/ { 1,0,1,28, "-01.234,56 DM|-01.234,56 DEM"},
#else
/*exp*/ { 1,0,1,29, "-01.234,56 DM|-01.234,56 DEM "},
#endif
}, },
{ {
/* #12 */ /* #12 */
/*inp*/ { 32, "%^#5n|%^#5i", -1234.561 }, /*inp*/ { 32, "%^#5n|%^#5i", -1234.561 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,26, "- 1234,56 DM|- 1234,56 DEM"}, /*exp*/ { 1,0,1,26, "- 1234,56 DM|- 1234,56 DEM"},
#else
/*exp*/ { 1,0,1,27, "- 1234,56 DM|- 1234,56 DEM "},
#endif
}, },
{ {
/* #13 */ /* #13 */
/*inp*/ { 32, "%#5.0n|%#5.0i", 1234.444 }, /*inp*/ { 32, "%#5.0n|%#5.0i", 1234.444 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,22, " 1.234 DM| 1.234 DEM" }, /*exp*/ { 1,0,1,22, " 1.234 DM| 1.234 DEM" },
#else
/*exp*/ { 1,0,1,23, " 1.234 DM| 1.234 DEM " },
#endif
}, },
{ {
/* #14 */ /* #14 */
/*inp*/ { 32, "%#5.0n|%#5.4i", -1234.555 }, /*inp*/ { 32, "%#5.0n|%#5.4i", -1234.555 },
#ifdef NO_WAIVER
/*exp*/ { 1,0,1,27, "- 1.235 DM|- 1.234,5550 DEM"}, /*exp*/ { 1,0,1,27, "- 1.235 DM|- 1.234,5550 DEM"},
#else
/*exp*/ { 1,0,1,28, "- 1.235 DM|- 1.234,5550 DEM "},
#endif
}, },
{ {
/* #15 */ /* #15 */

View File

@ -56,15 +56,15 @@ de_DE.ISO-8859-1 %+n -1234.56 -1.234,56 DM
de_DE.ISO-8859-1 %(n 1234.56 1.234,56 DM de_DE.ISO-8859-1 %(n 1234.56 1.234,56 DM
de_DE.ISO-8859-1 %(n -1234.56 (1.234,56 DM) de_DE.ISO-8859-1 %(n -1234.56 (1.234,56 DM)
de_DE.ISO-8859-1 %^n 1234.56 1234,56 DM de_DE.ISO-8859-1 %^n 1234.56 1234,56 DM
de_DE.ISO-8859-1 %i 1.23 1,23 DEM de_DE.ISO-8859-1 %i 1.23 1,23 DEM
de_DE.ISO-8859-1 %i -1.23 -1,23 DEM de_DE.ISO-8859-1 %i -1.23 -1,23 DEM
de_DE.ISO-8859-1 %i 1234.56 1.234,56 DEM de_DE.ISO-8859-1 %i 1234.56 1.234,56 DEM
de_DE.ISO-8859-1 %^i 1234.56 1234,56 DEM de_DE.ISO-8859-1 %^i 1234.56 1234,56 DEM
de_DE.ISO-8859-1 %+i 1234.56 1.234,56 DEM de_DE.ISO-8859-1 %+i 1234.56 1.234,56 DEM
de_DE.ISO-8859-1 %+i -1234.56 -1.234,56 DEM de_DE.ISO-8859-1 %+i -1234.56 -1.234,56 DEM
de_DE.ISO-8859-1 %(i 1234.56 1.234,56 DEM de_DE.ISO-8859-1 %(i 1234.56 1.234,56 DEM
de_DE.ISO-8859-1 %(i -1234.56 (1.234,56 DEM ) de_DE.ISO-8859-1 %(i -1234.56 (1.234,56 DEM)
de_DE.ISO-8859-1 %^i 1234.56 1234,56 DEM de_DE.ISO-8859-1 %^i 1234.56 1234,56 DEM
de_DE.ISO-8859-1 %#5n 123.45 123,45 DM de_DE.ISO-8859-1 %#5n 123.45 123,45 DM
de_DE.ISO-8859-1 %#5n -123.45 - 123,45 DM de_DE.ISO-8859-1 %#5n -123.45 - 123,45 DM
de_DE.ISO-8859-1 %#5n 3456.781 3.456,78 DM de_DE.ISO-8859-1 %#5n 3456.781 3.456,78 DM

View File

@ -52,6 +52,14 @@
out_char (*_s++); \ out_char (*_s++); \
} while (0) } while (0)
#define out_nstring(String, N) \
do { \
int _n = (N); \
const char *_s = (String); \
while (_n-- > 0) \
out_char (*_s++); \
} while (0)
#define to_digit(Ch) ((Ch) - '0') #define to_digit(Ch) ((Ch) - '0')
@ -136,9 +144,11 @@ __strfmon_l (char *s, size_t maxsize, __locale_t loc, const char *format, ...)
const char *other_sign_string; const char *other_sign_string;
int done; int done;
const char *currency_symbol; const char *currency_symbol;
size_t currency_symbol_len;
int width; int width;
char *startp; char *startp;
const void *ptr; const void *ptr;
char space_char;
/* Process all character which do not introduce a format /* Process all character which do not introduce a format
specification. */ specification. */
@ -294,6 +304,8 @@ __strfmon_l (char *s, size_t maxsize, __locale_t loc, const char *format, ...)
{ {
case 'i': /* Use international currency symbol. */ case 'i': /* Use international currency symbol. */
currency_symbol = _NL_CURRENT (LC_MONETARY, INT_CURR_SYMBOL); currency_symbol = _NL_CURRENT (LC_MONETARY, INT_CURR_SYMBOL);
currency_symbol_len = 3;
space_char = currency_symbol[3];
if (right_prec == -1) if (right_prec == -1)
{ {
if (*_NL_CURRENT (LC_MONETARY, INT_FRAC_DIGITS) == CHAR_MAX) if (*_NL_CURRENT (LC_MONETARY, INT_FRAC_DIGITS) == CHAR_MAX)
@ -304,6 +316,8 @@ __strfmon_l (char *s, size_t maxsize, __locale_t loc, const char *format, ...)
break; break;
case 'n': /* Use national currency symbol. */ case 'n': /* Use national currency symbol. */
currency_symbol = _NL_CURRENT (LC_MONETARY, CURRENCY_SYMBOL); currency_symbol = _NL_CURRENT (LC_MONETARY, CURRENCY_SYMBOL);
currency_symbol_len = strlen (currency_symbol);
space_char = ' ';
if (right_prec == -1) if (right_prec == -1)
{ {
if (*_NL_CURRENT (LC_MONETARY, FRAC_DIGITS) == CHAR_MAX) if (*_NL_CURRENT (LC_MONETARY, FRAC_DIGITS) == CHAR_MAX)
@ -424,14 +438,14 @@ __strfmon_l (char *s, size_t maxsize, __locale_t loc, const char *format, ...)
preceding the value */ preceding the value */
if (cs_precedes) if (cs_precedes)
{ {
left_bytes += strlen (currency_symbol); left_bytes += currency_symbol_len;
if (sep_by_space != 0) if (sep_by_space != 0)
++left_bytes; ++left_bytes;
} }
if (other_cs_precedes) if (other_cs_precedes)
{ {
other_left_bytes += strlen (currency_symbol); other_left_bytes += currency_symbol_len;
if (other_sep_by_space != 0) if (other_sep_by_space != 0)
++other_left_bytes; ++other_left_bytes;
} }
@ -491,7 +505,7 @@ __strfmon_l (char *s, size_t maxsize, __locale_t loc, const char *format, ...)
if (sign_posn == 4) if (sign_posn == 4)
{ {
if (sep_by_space == 2) if (sep_by_space == 2)
out_char (' '); out_char (space_char);
out_string (sign_string); out_string (sign_string);
if (sep_by_space == 1) if (sep_by_space == 1)
/* POSIX.2 and SUS are not clear on this case, but C99 /* POSIX.2 and SUS are not clear on this case, but C99
@ -500,7 +514,7 @@ __strfmon_l (char *s, size_t maxsize, __locale_t loc, const char *format, ...)
} }
else else
if (sep_by_space == 1) if (sep_by_space == 1)
out_char (' '); out_char (space_char);
} }
} }
else else
@ -580,8 +594,8 @@ __strfmon_l (char *s, size_t maxsize, __locale_t loc, const char *format, ...)
|| (sign_posn == 2 && sep_by_space == 1) || (sign_posn == 2 && sep_by_space == 1)
|| (sign_posn == 1 && sep_by_space == 1) || (sign_posn == 1 && sep_by_space == 1)
|| (sign_posn == 0 && sep_by_space == 1)) || (sign_posn == 0 && sep_by_space == 1))
out_char (' '); out_char (space_char);
out_string (currency_symbol); out_nstring (currency_symbol, currency_symbol_len);
if (sign_posn == 4) if (sign_posn == 4)
{ {
if (sep_by_space == 2) if (sep_by_space == 2)