From c0ea17cc7c93a43a4d92123dea4c7118c532521a Mon Sep 17 00:00:00 2001 From: Paolo Carlini Date: Tue, 30 Dec 2003 10:43:54 +0000 Subject: 2003-12-30 Paolo Carlini PR libstdc++/13369 * include/bits/locale_facets.tcc (__verify_grouping): Fix to deal properly with __grouping_tmp.size() > __grouping.size(). * testsuite/22_locale/num_get/get/char/13.cc: New. * testsuite/22_locale/num_get/get/wchar_t/13.cc: Ditto. git-svn-id: https://gcc.gnu.org/svn/gcc/trunk@75227 138bc75d-0d04-0410-961f-82ee72b054a4 --- libstdc++-v3/ChangeLog | 9 +++ libstdc++-v3/include/bits/locale_facets.tcc | 21 +++--- .../testsuite/22_locale/num_get/get/char/13.cc | 79 ++++++++++++++++++++++ .../testsuite/22_locale/num_get/get/wchar_t/13.cc | 79 ++++++++++++++++++++++ 4 files changed, 177 insertions(+), 11 deletions(-) create mode 100644 libstdc++-v3/testsuite/22_locale/num_get/get/char/13.cc create mode 100644 libstdc++-v3/testsuite/22_locale/num_get/get/wchar_t/13.cc (limited to 'libstdc++-v3') diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 5c8ca195a56..5add00b67ea 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,3 +1,12 @@ +2003-12-30 Paolo Carlini + + PR libstdc++/13369 + * include/bits/locale_facets.tcc (__verify_grouping): + Fix to deal properly with __grouping_tmp.size() > + __grouping.size(). + * testsuite/22_locale/num_get/get/char/13.cc: New. + * testsuite/22_locale/num_get/get/wchar_t/13.cc: Ditto. + 2003-12-29 Jerry Quinn * include/bits/locale_facets.h (ctype.narrow,widen): Add cast. diff --git a/libstdc++-v3/include/bits/locale_facets.tcc b/libstdc++-v3/include/bits/locale_facets.tcc index 265c1437eba..40b612c1dde 100644 --- a/libstdc++-v3/include/bits/locale_facets.tcc +++ b/libstdc++-v3/include/bits/locale_facets.tcc @@ -2263,23 +2263,22 @@ namespace std bool __verify_grouping(const basic_string<_CharT>& __grouping, const basic_string<_CharT>& __grouping_tmp) - { - size_t __i = 0; - size_t __j = 0; - const size_t __len = __grouping.size(); - const size_t __n = __grouping_tmp.size(); + { + const size_t __n = __grouping_tmp.size() - 1; + const size_t __min = std::min(__n, __grouping.size() - 1); + size_t __i = __n; bool __test = true; - + // Parsed number groupings have to match the // numpunct::grouping string exactly, starting at the // right-most point of the parsed sequence of elements ... - while (__test && __i < __n - 1) - for (__j = 0; __test && __j < __len && __i < __n - 1; ++__j, ++__i) - __test = __grouping[__j] == __grouping_tmp[__n - __i - 1]; + for (size_t __j = 0; __j < __min && __test; --__i, ++__j) + __test = __grouping_tmp[__i] == __grouping[__j]; + for (; __i && __test; --__i) + __test = __grouping_tmp[__i] == __grouping[__min]; // ... but the last parsed grouping can be <= numpunct // grouping. - __j == __len ? __j = 0 : __j; - __test &= __grouping[__j] >= __grouping_tmp[__n - __i - 1]; + __test &= __grouping_tmp[0] <= __grouping[__min]; return __test; } diff --git a/libstdc++-v3/testsuite/22_locale/num_get/get/char/13.cc b/libstdc++-v3/testsuite/22_locale/num_get/get/char/13.cc new file mode 100644 index 00000000000..63cb008e850 --- /dev/null +++ b/libstdc++-v3/testsuite/22_locale/num_get/get/char/13.cc @@ -0,0 +1,79 @@ +// 2003-12-30 Paolo Carlini + +// Copyright (C) 2003 Free Software Foundation +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 2, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING. If not, write to the Free +// Software Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, +// USA. + +// 22.2.2.1.1 num_get members + +#include +#include +#include + +struct Punct1: std::numpunct +{ std::string do_grouping() const { return "\003\002\001"; } }; + +struct Punct2: std::numpunct +{ std::string do_grouping() const { return "\001\003"; } }; + +// libstdc++/13369 +void test01() +{ + using namespace std; + typedef istreambuf_iterator iterator_type; + + bool test __attribute__((unused)) = true; + + istringstream iss1, iss2; + iss1.imbue(locale(iss1.getloc(), static_cast*>(new Punct1))); + iss2.imbue(locale(iss2.getloc(), static_cast*>(new Punct2))); + const num_get& ng1 = use_facet >(iss1.getloc()); + const num_get& ng2 = use_facet >(iss2.getloc()); + + ios_base::iostate err = ios_base::goodbit; + iterator_type end; + long l = 0l; + long l1 = 12345678l; + double d = 0.0; + double d1 = 1234567.0; + double d2 = 123456.0; + + iss1.str("1,2,3,45,678"); + err = ios_base::goodbit; + end = ng1.get(iss1.rdbuf(), 0, iss1, err, l); + VERIFY( err == ios_base::eofbit ); + VERIFY( l == l1 ); + + iss2.str("123,456,7.0"); + err = ios_base::goodbit; + end = ng2.get(iss2.rdbuf(), 0, iss2, err, d); + VERIFY( err == ios_base::eofbit ); + VERIFY( d == d1 ); + + iss2.str("12,345,6.0"); + iss2.clear(); + err = ios_base::goodbit; + end = ng2.get(iss2.rdbuf(), 0, iss2, err, d); + VERIFY( err == ios_base::eofbit ); + VERIFY( d == d2 ); +} + +int main() +{ + test01(); + return 0; +} diff --git a/libstdc++-v3/testsuite/22_locale/num_get/get/wchar_t/13.cc b/libstdc++-v3/testsuite/22_locale/num_get/get/wchar_t/13.cc new file mode 100644 index 00000000000..217695957b8 --- /dev/null +++ b/libstdc++-v3/testsuite/22_locale/num_get/get/wchar_t/13.cc @@ -0,0 +1,79 @@ +// 2003-12-30 Paolo Carlini + +// Copyright (C) 2003 Free Software Foundation +// +// This file is part of the GNU ISO C++ Library. This library is free +// software; you can redistribute it and/or modify it under the +// terms of the GNU General Public License as published by the +// Free Software Foundation; either version 2, or (at your option) +// any later version. + +// This library is distributed in the hope that it will be useful, +// but WITHOUT ANY WARRANTY; without even the implied warranty of +// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +// GNU General Public License for more details. + +// You should have received a copy of the GNU General Public License along +// with this library; see the file COPYING. If not, write to the Free +// Software Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, +// USA. + +// 22.2.2.1.1 num_get members + +#include +#include +#include + +struct Punct1: std::numpunct +{ std::string do_grouping() const { return "\003\002\001"; } }; + +struct Punct2: std::numpunct +{ std::string do_grouping() const { return "\001\003"; } }; + +// libstdc++/13369 +void test01() +{ + using namespace std; + typedef istreambuf_iterator iterator_type; + + bool test __attribute__((unused)) = true; + + wistringstream iss1, iss2; + iss1.imbue(locale(iss1.getloc(), static_cast*>(new Punct1))); + iss2.imbue(locale(iss2.getloc(), static_cast*>(new Punct2))); + const num_get& ng1 = use_facet >(iss1.getloc()); + const num_get& ng2 = use_facet >(iss2.getloc()); + + ios_base::iostate err = ios_base::goodbit; + iterator_type end; + long l = 0l; + long l1 = 12345678l; + double d = 0.0; + double d1 = 1234567.0; + double d2 = 123456.0; + + iss1.str(L"1,2,3,45,678"); + err = ios_base::goodbit; + end = ng1.get(iss1.rdbuf(), 0, iss1, err, l); + VERIFY( err == ios_base::eofbit ); + VERIFY( l == l1 ); + + iss2.str(L"123,456,7.0"); + err = ios_base::goodbit; + end = ng2.get(iss2.rdbuf(), 0, iss2, err, d); + VERIFY( err == ios_base::eofbit ); + VERIFY( d == d1 ); + + iss2.str(L"12,345,6.0"); + iss2.clear(); + err = ios_base::goodbit; + end = ng2.get(iss2.rdbuf(), 0, iss2, err, d); + VERIFY( err == ios_base::eofbit ); + VERIFY( d == d2 ); +} + +int main() +{ + test01(); + return 0; +} -- cgit v1.2.3