Tests fixes and debug vector pendantic capacity management change

François Dumont frs.dumont@gmail.com
Sat Dec 20 16:29:00 GMT 2014


Hi

     Following a discussion on Reflector that has been reported to me, 
people are complaining about management of the vector capacity in the 
pedantic debug mode. Even if it is not crystal clear in the Standard 
there seems to be a consensus that clear shouldn't impact it. So here is 
a simple patch to stop resetting _M_guaranteed_capacity in this method.

     As I run all tests in debug mode I also fix another problem. In 
testsuite/23_containers/vector/bool/allocator/swap.cc == and != 
propagating_allocator operators must be within __gnu_test namespace in 
order to be considered thanks to ADL when in debug mode.

     There are also 2 tests that XPASS:
XPASS: experimental/string_view/element_access/char/2.cc execution test
XPASS: experimental/string_view/element_access/wchar_t/2.cc execution test

     Debug assertions has been commented about because not compatible 
with constexpr. Do we keep them this way ?

2014-12-20  François Dumont  <fdumont@gcc.gnu.org>

     * include/debug/vector (std::__debug::vector<>::clear()): Do not reset
     guaranteed capacity.
     * testsuite/23_containers/vector/bool/allocator/swap.cc: Move
     propagating_allocator equality and inequality operators to __gnu_test
     namespace.

François

-------------- next part --------------
A non-text attachment was scrubbed...
Name: debug.patch
Type: text/x-patch
Size: 1163 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/libstdc++/attachments/20141220/4e80ea49/attachment.bin>


More information about the Libstdc++ mailing list