Re: fix libstdc++/52433

2012-03-08 Thread Jonathan Wakely
On 8 March 2012 10:22, Paolo Carlini wrote: > > By the way, this morning it occurred to me that we should probably also have > debug-mode checks for self move-assignment (not just in _Safe_iterator)?!? Yes, nice idea, the library is allowed to assume it doesn't happen, but we can and should check

Re: fix libstdc++/52433

2012-03-08 Thread Paolo Carlini
On 03/08/2012 02:06 AM, Jonathan Wakely wrote: On 4 March 2012 12:56, Jonathan Wakely wrote: PR libstdc++/52433 * include/debug/safe_iterator.h (_Safe_iterator): Add move constructor and move assignment operator. * testsuite/23_containers/vector/debug/52433.cc: Ne

Re: fix libstdc++/52433

2012-03-07 Thread Jonathan Wakely
On 4 March 2012 12:56, Jonathan Wakely wrote: >        PR libstdc++/52433 >        * include/debug/safe_iterator.h (_Safe_iterator): Add move >        constructor and move assignment operator. >        * testsuite/23_containers/vector/debug/52433.cc: New. > > Tested 'make check check-debug' on x86_

fix libstdc++/52433

2012-03-04 Thread Jonathan Wakely
PR libstdc++/52433 * include/debug/safe_iterator.h (_Safe_iterator): Add move constructor and move assignment operator. * testsuite/23_containers/vector/debug/52433.cc: New. Tested 'make check check-debug' on x86_64 and committed to trunk. I plan to fix this for 4.