From 1638f5c9d007cf4e1a19a9a44b2f24a311c6727a Mon Sep 17 00:00:00 2001 From: Paolo Carlini Date: Tue, 14 Oct 2003 19:15:27 +0200 Subject: [PATCH] re PR libstdc++/11480 (std::unique calls predicate too many times) 2003-10-14 Paolo Carlini PR libstdc++/11480 * include/bits/stl_algo.h (unique): Fix. * testsuite/25_algorithms/unique.cc: Move to unique/1.cc. * testsuite/25_algorithms/unique/11480.cc: New, from the PR. * testsuite/25_algorithms/unique/2.cc: New. From-SVN: r72478 --- libstdc++-v3/ChangeLog | 8 ++ libstdc++-v3/include/bits/stl_algo.h | 24 ++++- .../25_algorithms/{unique.cc => unique/1.cc} | 6 +- .../testsuite/25_algorithms/unique/11480.cc | 46 +++++++++ .../testsuite/25_algorithms/unique/2.cc | 95 +++++++++++++++++++ 5 files changed, 173 insertions(+), 6 deletions(-) rename libstdc++-v3/testsuite/25_algorithms/{unique.cc => unique/1.cc} (90%) create mode 100644 libstdc++-v3/testsuite/25_algorithms/unique/11480.cc create mode 100644 libstdc++-v3/testsuite/25_algorithms/unique/2.cc diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index b5381140aac..481dd3729cc 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,3 +1,11 @@ +2003-10-14 Paolo Carlini + + PR libstdc++/11480 + * include/bits/stl_algo.h (unique): Fix. + * testsuite/25_algorithms/unique.cc: Move to unique/1.cc. + * testsuite/25_algorithms/unique/11480.cc: New, from the PR. + * testsuite/25_algorithms/unique/2.cc: New. + 2003-10-14 Paolo Carlini * src/localename.cc (_M_replace_categories, M_replace_facet): diff --git a/libstdc++-v3/include/bits/stl_algo.h b/libstdc++-v3/include/bits/stl_algo.h index 062550cc98b..70ef2199536 100644 --- a/libstdc++-v3/include/bits/stl_algo.h +++ b/libstdc++-v3/include/bits/stl_algo.h @@ -1268,8 +1268,18 @@ namespace std __glibcxx_function_requires(_EqualityComparableConcept< typename iterator_traits<_ForwardIterator>::value_type>) + // Skip the beginning, if already unique. __first = std::adjacent_find(__first, __last); - return std::unique_copy(__first, __last, __first); + if (__first == __last) + return __last; + + // Do the real copy work. + _ForwardIterator __dest = __first; + ++__first; + while (++__first != __last) + if (!(*__dest == *__first)) + *++__dest = *__first; + return ++__dest; } /** @@ -1297,8 +1307,18 @@ namespace std typename iterator_traits<_ForwardIterator>::value_type, typename iterator_traits<_ForwardIterator>::value_type>) + // Skip the beginning, if already unique. __first = std::adjacent_find(__first, __last, __binary_pred); - return std::unique_copy(__first, __last, __first, __binary_pred); + if (__first == __last) + return __last; + + // Do the real copy work. + _ForwardIterator __dest = __first; + ++__first; + while (++__first != __last) + if (!__binary_pred(*__dest, *__first)) + *++__dest = *__first; + return ++__dest; } /** diff --git a/libstdc++-v3/testsuite/25_algorithms/unique.cc b/libstdc++-v3/testsuite/25_algorithms/unique/1.cc similarity index 90% rename from libstdc++-v3/testsuite/25_algorithms/unique.cc rename to libstdc++-v3/testsuite/25_algorithms/unique/1.cc index 6a4d7f23b24..6c383bb4af4 100644 --- a/libstdc++-v3/testsuite/25_algorithms/unique.cc +++ b/libstdc++-v3/testsuite/25_algorithms/unique/1.cc @@ -1,6 +1,6 @@ // 2001-07-19 Peter Schmid -// Copyright (C) 2001 Free Software Foundation, Inc. +// Copyright (C) 2001, 2002, 2003 Free Software Foundation, Inc. // // 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 @@ -18,13 +18,11 @@ // Software Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, // USA. -// 25.2.8 Unique - +// 25.2.8 [lib.alg.unique] Unique #include #include - int main() { using namespace std; diff --git a/libstdc++-v3/testsuite/25_algorithms/unique/11480.cc b/libstdc++-v3/testsuite/25_algorithms/unique/11480.cc new file mode 100644 index 00000000000..6ac9f685790 --- /dev/null +++ b/libstdc++-v3/testsuite/25_algorithms/unique/11480.cc @@ -0,0 +1,46 @@ +// Copyright (C) 2003 Free Software Foundation, Inc. +// +// 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. + +#include +#include + +int a[10] = { 1, 2, 3, 3, 4, 5, 5, 6, 7, 9 }; + +static int compare_count = 0; + +bool compare(int a, int b) +{ + compare_count++; + return a == b; +} + +// libstdc++/11480 +void test01() +{ + bool test __attribute__((unused)) = true; + + std::unique(a, a+10, compare); + VERIFY( compare_count == 9 ); +} + +int +main() +{ + test01(); + return 0; +} diff --git a/libstdc++-v3/testsuite/25_algorithms/unique/2.cc b/libstdc++-v3/testsuite/25_algorithms/unique/2.cc new file mode 100644 index 00000000000..55920fbe388 --- /dev/null +++ b/libstdc++-v3/testsuite/25_algorithms/unique/2.cc @@ -0,0 +1,95 @@ +// 2003-10-14 Paolo Carlini + +// Copyright (C) 2003 Free Software Foundation, Inc. +// +// 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. + +// 25.2.8 [lib.alg.unique] Unique + +#include +#include +#include +#include + +const int T1[] = {1, 4, 4, 6, 1, 2, 2, 3, 1, 6, 6, 6, 5, 7, 5, 4, 4}; +const int T2[] = {1, 1, 1, 2, 2, 1, 1, 7, 6, 6, 7, 8, 8, 8, 8, 9, 9}; +const int N = sizeof(T1) / sizeof(int); + +const int A1[] = {1, 4, 6, 1, 2, 3, 1, 6, 5, 7, 5, 4}; +const int A2[] = {1, 4, 4, 6, 6, 6, 6, 7}; +const int A3[] = {1, 1, 1}; + +const int B1[] = {1, 2, 1, 7, 6, 7, 8, 9}; +const int B2[] = {1, 1, 1, 2, 2, 7, 7, 8, 8, 8, 8, 9, 9}; +const int B3[] = {9, 9, 8, 8, 8, 8, 7, 6, 6, 1, 1, 1, 1, 1}; + +void test01() +{ + bool test __attribute__((unused)) = true; + using namespace std; + + list::iterator pos; + + list coll(T1, T1 + N); + pos = unique(coll.begin(), coll.end()); + VERIFY( equal(coll.begin(), pos, A1) ); + + list coll2(T2, T2 + N); + pos = unique(coll2.begin(), coll2.end()); + VERIFY( equal(coll2.begin(), pos, B1) ); +} + +void test02() +{ + bool test __attribute__((unused)) = true; + using namespace std; + + list::iterator pos; + + list coll(T1, T1 + N); + pos = unique(coll.begin(), coll.end(), greater()); + VERIFY( equal(coll.begin(), pos, A2) ); + + list coll2(T2, T2 + N); + pos = unique(coll2.begin(), coll2.end(), greater()); + VERIFY( equal(coll2.begin(), pos, B2) ); +} + +void test03() +{ + bool test __attribute__((unused)) = true; + using namespace std; + + list::iterator pos; + + list coll(T1, T1 + N); + pos = unique(coll.begin(), coll.end(), less()); + VERIFY( equal(coll.begin(), pos, A3) ); + + list coll2(T2, T2 + N); + reverse(coll2.begin(), coll2.end()); + pos = unique(coll2.begin(), coll2.end(), less()); + VERIFY( equal(coll2.begin(), pos, B3) ); +} + +int main() +{ + test01(); + test02(); + test03(); + return 0; +} -- 2.30.2