Re: [patch] debug mode

[François Dumont <francois dot cppdevs at free dot fr>]

Hi

    As I only receive mailing list entries in a digest form I miss Bo 
and you answer about __f and __l. As you seem to prefer that I do not 
change the name of variable to respect line width limit here is the 
patch again. I even change some __f, __l in __first, __last for 
consistency accross the lib.

    Here is also the ChangeLog entry I miss in my previous post.

2010-08-24  François Dumont <francois.cppdevs@free.fr>

        * include/bits/stl_algobase.h: _Iter_base struct moved...
        * include/bits/stl_iterator_base_types.h: ...here.
        * include/debug/functions.h: Fix behavior of __check_valid_range,
        __check_string, __check_sorted debug functions that used to 
depend on
        _GLIBCXX_DEBUG macro. Include formatter.h and use formatting 
macros
        for a consistent debug result.
        * include/debug/formatter.h: Remove debug.h include, add 
__check_singular
        declaration.
        * include/debug/debug.h: Remove formatter.h include that is now
        included by functions.h.
        * include/debug/safe_iterator.h: _Safe_iterator::_Base_Iterator
        renamed in interator_type to make it usable by _Iter_base helper
        struct. __gnu_debug::__base introduced.
        * include/debug/set.h: Use __gnu_debug::__base once iterator range
        valided.
        * include/debug/unordered_map: Same.
        * include/debug/multiset.h: Same.
        * include/debug/vector: Same.
        * include/debug/unordered_set: Same.
        * include/debug/deque: Same.
        * include/debug/map.h: Same.
        * include/debug/string: Same.
        * include/debug/list: Same.
        * include/debug/multimap.h: Same.
        * testsuite/23_containers/util/debug/assign_neg.h: New test 
cases on
        debug checks performed on container assign operation.
        * testsuite/23_containers/util/debug/construct_neg.h: New test 
cases
        on debug checks on constructors.
        * testsuite/23_containers/util/debug/insert_neg.h: New test 
cases on
        debug checks performed on container insert operations.
        * testsuite/23_containers/unordered_map/debug/cont_traits.h,
        debug_cont_traits.h, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        unordered_map.
        * testsuite/23_containers/multimap/debug/cont_traits.h,
        debug_cont_traits.h, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        multimap.
        * testsuite/23_containers/set/debug/cont_traits.h,
        debug_cont_traits.h, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        set.
        * testsuite/23_containers/unordered_multimap/debug/cont_traits.h,
        debug_cont_traits.h, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        unordered_multimap.
        * testsuite/23_containers/unordered_set/debug/cont_traits.h,
        debug_cont_traits.h, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        unordered_set.
        * testsuite/23_containers/multiset/debug/cont_traits.h,
        debug_cont_traits.h, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        multiset.
        * testsuite/23_containers/unordered_multiset/debug/cont_traits.h,
        debug_cont_traits.h, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        unordered_multiset.
        * testsuite/23_containers/map/debug/cont_traits.h,
        debug_cont_traits.h, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        map.
        * testsuite/23_containers/vector/debug/cont_traits.h,
        debug_cont_traits.h, assign1_neg.cc, assign2_neg.cc, 
assign3_neg.cc,
        assign4_neg.cc, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        vector.
        * testsuite/23_containers/deque/debug/cont_traits.h,
        debug_cont_traits.h, assign1_neg.cc, assign2_neg.cc, 
assign3_neg.cc,
        assign4_neg.cc, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        deque.
        * testsuite/23_containers/list/debug/cont_traits.h,
        debug_cont_traits.h, assign1_neg.cc, assign2_neg.cc, 
assign3_neg.cc,
        assign4_neg.cc, construct1_neg.cc, construct2_neg.cc,
        construct3_neg.cc, construct4_neg.cc, insert1_neg.cc, 
insert2_neg.cc,
        insert3_neg.cc, insert4_neg.cc: New, debug test cases on
        list.
        * 
testsuite/performance/23_containers/range_construct/list_construct1.cc:
        New, validate performance impact of the patch on the debug mode
        * 
testsuite/performance/23_containers/range_construct/list_construct2.cc:
        Idem
        * 
testsuite/performance/23_containers/range_construct/vector_construct.cc:
        Idem
        * 
testsuite/performance/23_containers/range_construct/deque_construct.cc:
        Idem

Bests

Attachment: debug.patch
Description: Text document