This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PR libstdc++/60587
- From: Jonathan Wakely <jwakely at redhat dot com>
- To: libstdc++ at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
- Cc: "frs dot dumont at gmail dot com" <frs dot dumont at gmail dot com>
- Date: Fri, 21 Mar 2014 18:56:02 +0000
- Subject: Re: PR libstdc++/60587
- Authentication-results: sourceware.org; auth=none
- References: <20140319213903 dot GA23191 at redhat dot com> <20140319222844 dot GA19403 at redhat dot com> <9E706DC1-54BA-42C7-B399-60CF3CFD0602 at oracle dot com> <20140320202932 dot GB19403 at redhat dot com>
On 20/03/14 20:29 +0000, Jonathan Wakely wrote:
Everything passes for "make check check-debug", so I plan to commit it
tomorrow, unless anyone points out a problem.
Committed to trunk.
PR libstdc++/60587
* include/debug/functions.h (_Is_contiguous_sequence): Define.
(__foreign_iterator): Accept additional iterator. Do not dispatch on
iterator category.
(__foreign_iterator_aux2): Likewise. Add overload for iterators
from different types of debug container. Use _Is_contiguous_sequence
instead of is_lvalue_reference.
(__foreign_iterator_aux3): Accept additional iterator. Avoid
dereferencing past-the-end iterator.
(__foreign_iterator_aux4): Use const value_type* instead of
potentially user-defined const_pointer type.
* include/debug/macros.h (__glibcxx_check_insert_range): Fix comment
and pass end iterator to __gnu_debug::__foreign_iterator.
(__glibcxx_check_insert_range_after): Likewise.
(__glibcxx_check_max_load_factor): Fix comment.
* include/debug/vector (_Is_contiguous_sequence): Define partial
specializations.
* testsuite/23_containers/vector/debug/57779_neg.cc: Remove
-std=gnu++11 option and unused header.
* testsuite/23_containers/vector/debug/60587.cc: New.
* testsuite/23_containers/vector/debug/60587_neg.cc: New.