forward_list debug swap

François Dumont francois.cppdevs@free.fr
Wed Jan 5 21:41:00 GMT 2011


This is the patch finally applied.

2011-01-05  François Dumont <francois.cppdevs@free.fr>

         * include/debug/safe_base.h (_Safe_iterator_base::_M_unlink): New.
         * include/src/debug.cc: Use latter
         * include/debug/forward_list (forward_list<>::_M_swap):  Fix to
         correctly handle before_begin iterators.
         * testsuite/23_containers/forward_list/debug/swap.cc: Remove now
         useless _GLIBCXX_DEBUG checks.

     I realized that _M_version fields in _Safe_iterator_base and 
_Safe_sequence_base are useless for a sequence like the forward_list. I 
will consider this optimization for a future (binary incompatible) 
evolution of the debug mode.

Regards

On 01/03/2011 10:07 PM, Paolo Carlini wrote:
> Ok, but please properly format the code: no overlong lines, newline
> after the return type of the function and after the double colon for
> member functions.
>
> Paolo.
>
>    

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: forward_list_debug_swap.patch
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20110105/9adfb1f9/attachment.ksh>


More information about the Gcc-patches mailing list