PR libstdc++/87642 handle multibyte thousands separators from libc

If a locale's THOUSANDS_SEP or MON_THOUSANDS_SEP string is not a
single character we either need to narrow it to a single char or
ignore it (and therefore disable digit grouping for that facet).

	PR libstdc++/87642
	* config/locale/gnu/monetary_members.cc
	(moneypunct<char, true>::_M_initialize_moneypunct): Use
	__narrow_multibyte_chars to convert multibyte thousands separators
	to a single char.
	* config/locale/gnu/numeric_members.cc
	(numpunct<char>::_M_initialize_numpunct): Likewise.
	(__narrow_multibyte_chars): New function.

From-SVN: r265286
This commit is contained in:
Jonathan Wakely 2018-10-18 20:57:25 +01:00 committed by Jonathan Wakely
parent e8ce849a48
commit c0ace69ec6
3 changed files with 76 additions and 4 deletions

View File

@ -1,5 +1,14 @@
2018-10-18 Jonathan Wakely <jwakely@redhat.com> 2018-10-18 Jonathan Wakely <jwakely@redhat.com>
PR libstdc++/87642
* config/locale/gnu/monetary_members.cc
(moneypunct<char, true>::_M_initialize_moneypunct): Use
__narrow_multibyte_chars to convert multibyte thousands separators
to a single char.
* config/locale/gnu/numeric_members.cc
(numpunct<char>::_M_initialize_numpunct): Likewise.
(__narrow_multibyte_chars): New function.
PR libstdc++/87641 PR libstdc++/87641
* include/bits/valarray_array.h (__valarray_sum): Use first element * include/bits/valarray_array.h (__valarray_sum): Use first element
to initialize accumulator instead of value-initializing it. to initialize accumulator instead of value-initializing it.

View File

@ -207,6 +207,8 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
} }
#endif #endif
extern char __narrow_multibyte_chars(const char* s, __locale_t cloc);
template<> template<>
void void
moneypunct<char, true>::_M_initialize_moneypunct(__c_locale __cloc, moneypunct<char, true>::_M_initialize_moneypunct(__c_locale __cloc,
@ -241,8 +243,13 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
// Named locale. // Named locale.
_M_data->_M_decimal_point = *(__nl_langinfo_l(__MON_DECIMAL_POINT, _M_data->_M_decimal_point = *(__nl_langinfo_l(__MON_DECIMAL_POINT,
__cloc)); __cloc));
_M_data->_M_thousands_sep = *(__nl_langinfo_l(__MON_THOUSANDS_SEP, const char* thousands_sep = __nl_langinfo_l(__MON_THOUSANDS_SEP,
__cloc)); __cloc);
if (thousands_sep[0] != '\0' && thousands_sep[1] != '\0')
_M_data->_M_thousands_sep = __narrow_multibyte_chars(thousands_sep,
__cloc);
else
_M_data->_M_thousands_sep = *thousands_sep;
// Check for NULL, which implies no fractional digits. // Check for NULL, which implies no fractional digits.
if (_M_data->_M_decimal_point == '\0') if (_M_data->_M_decimal_point == '\0')

View File

@ -30,11 +30,62 @@
#include <locale> #include <locale>
#include <bits/c++locale_internal.h> #include <bits/c++locale_internal.h>
#include <iconv.h>
namespace std _GLIBCXX_VISIBILITY(default) namespace std _GLIBCXX_VISIBILITY(default)
{ {
_GLIBCXX_BEGIN_NAMESPACE_VERSION _GLIBCXX_BEGIN_NAMESPACE_VERSION
extern char __narrow_multibyte_chars(const char* s, __locale_t cloc);
// This file might be compiled twice, but we only want to define this once.
#if ! _GLIBCXX_USE_CXX11_ABI
char
__narrow_multibyte_chars(const char* s, __locale_t cloc)
{
const char* codeset = __nl_langinfo_l(CODESET, cloc);
if (!strcmp(codeset, "UTF-8"))
{
// optimize for some known cases
if (!strcmp(s, "\u202F")) // NARROW NO-BREAK SPACE
return ' ';
if (!strcmp(s, "\u2019")) // RIGHT SINGLE QUOTATION MARK
return '\'';
if (!strcmp(s, "\u066C")) // ARABIC THOUSANDS SEPARATOR
return '\'';
}
iconv_t cd = iconv_open("ASCII//TRANSLIT", codeset);
if (cd != (iconv_t)-1)
{
char c1;
size_t inbytesleft = strlen(s);
size_t outbytesleft = 1;
char* inbuf = const_cast<char*>(s);
char* outbuf = &c1;
size_t n = iconv(cd, &inbuf, &inbytesleft, &outbuf, &outbytesleft);
iconv_close(cd);
if (n != (size_t)-1)
{
cd = iconv_open(codeset, "ASCII");
if (cd != (iconv_t)-1)
{
char c2;
inbuf = &c1;
inbytesleft = 1;
outbuf = &c2;
outbytesleft = 1;
n = iconv(cd, &inbuf, &inbytesleft, &outbuf, &outbytesleft);
iconv_close(cd);
if (n != (size_t)-1)
return c2;
}
}
}
return '\0';
}
#endif
template<> template<>
void void
numpunct<char>::_M_initialize_numpunct(__c_locale __cloc) numpunct<char>::_M_initialize_numpunct(__c_locale __cloc)
@ -63,8 +114,13 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
// Named locale. // Named locale.
_M_data->_M_decimal_point = *(__nl_langinfo_l(DECIMAL_POINT, _M_data->_M_decimal_point = *(__nl_langinfo_l(DECIMAL_POINT,
__cloc)); __cloc));
_M_data->_M_thousands_sep = *(__nl_langinfo_l(THOUSANDS_SEP, const char* thousands_sep = __nl_langinfo_l(THOUSANDS_SEP, __cloc);
__cloc));
if (thousands_sep[0] != '\0' && thousands_sep[1] != '\0')
_M_data->_M_thousands_sep = __narrow_multibyte_chars(thousands_sep,
__cloc);
else
_M_data->_M_thousands_sep = *thousands_sep;
// Check for NULL, which implies no grouping. // Check for NULL, which implies no grouping.
if (_M_data->_M_thousands_sep == '\0') if (_M_data->_M_thousands_sep == '\0')