From 605d62e9de00b9f95a00202f66dfb03fe52ca523 Mon Sep 17 00:00:00 2001 From: Marshall Clow Date: Tue, 4 Jun 2019 15:18:46 +0000 Subject: [PATCH] No longer reject inputs when using a locale that has grouping information _and_ the input has no grouping characters at all. We continue to reject cases when the input has grouping characters in the wrong place. Fixes PR#28704 llvm-svn: 362508 --- libcxx/src/locale.cpp | 4 +++- .../locale.num.get/facet.num.get.members/get_long.pass.cpp | 4 ++-- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/libcxx/src/locale.cpp b/libcxx/src/locale.cpp index 00eb574..ca0a1c9 100644 --- a/libcxx/src/locale.cpp +++ b/libcxx/src/locale.cpp @@ -4379,7 +4379,9 @@ void __check_grouping(const string& __grouping, unsigned* __g, unsigned* __g_end, ios_base::iostate& __err) { - if (__grouping.size() != 0) +// if the grouping pattern is empty _or_ there are no grouping bits, then do nothing +// we always have at least a single entry in [__g, __g_end); the end of the input sequence + if (__grouping.size() >= 0 && __g_end - __g > 1) { reverse(__g, __g_end); const char* __ig = __grouping.data(); diff --git a/libcxx/test/std/localization/locale.categories/category.numeric/locale.num.get/facet.num.get.members/get_long.pass.cpp b/libcxx/test/std/localization/locale.categories/category.numeric/locale.num.get/facet.num.get.members/get_long.pass.cpp index 4212626..00b0c3a 100644 --- a/libcxx/test/std/localization/locale.categories/category.numeric/locale.num.get/facet.num.get.members/get_long.pass.cpp +++ b/libcxx/test/std/localization/locale.categories/category.numeric/locale.num.get/facet.num.get.members/get_long.pass.cpp @@ -162,14 +162,14 @@ int main(int, char**) ios.imbue(std::locale(std::locale(), new my_numpunct)); { v = -1; - const char str[] = "123"; + const char str[] = "123"; // no separators at all std::ios_base::iostate err = ios.goodbit; input_iterator iter = f.get(input_iterator(str), input_iterator(str+sizeof(str)), ios, err, v); assert(iter.base() == str+sizeof(str)-1); - assert(err == ios.failbit); + assert(err == ios.goodbit); assert(v == 123); } { -- 2.7.4