Re: copy/copy_backward/fill/fill_n/equal rework

[Jonathan Wakely]

On 10/12/19 15:19 +0000, Jonathan Wakely wrote:

On 09/12/19 10:32 +0100, François Dumont wrote:

After completing this work and running more tests I realized that the declaration of algos was still not ideal.

So here is another version where algos are not re-declare in stl_deque.h, I rather include stl_algobase.h in deque.tcc. The problem was spotted but another patch I am going to submit afterward.

OK for trunk (with a suitable ChangeLog).

Thanks for persisting with this, sorry it took so long.

A small fix for C++98 compat, I'll commit this to trunk shortly.



commit a9b209bd8032a36ad0a02d15358e5c5d699df9fa
Author: Jonathan Wakely <jwak...@redhat.com>
Date:   Fri Jan 10 15:02:07 2020 +0000

    libstdc++: Fix testcase for C++98 compatibility
    
            * testsuite/25_algorithms/equal/deque_iterators/1.cc: Don't use C++11
            initialization syntax.

diff --git a/libstdc++-v3/testsuite/25_algorithms/equal/deque_iterators/1.cc b/libstdc++-v3/testsuite/25_algorithms/equal/deque_iterators/1.cc
index be60db813cc..82bef051542 100644
--- a/libstdc++-v3/testsuite/25_algorithms/equal/deque_iterators/1.cc
+++ b/libstdc++-v3/testsuite/25_algorithms/equal/deque_iterators/1.cc
@@ -104,7 +104,7 @@ void test05()
 {
   using namespace std;
 
-  int a[] { 0, 1, 2, 3, 4 };
+  int a[] = { 0, 1, 2, 3, 4 };
   deque<int, __gnu_cxx::new_allocator<int> > d1(a, a + 5);
   deque<int, __gnu_cxx::malloc_allocator<int> > d2(a, a + 5);