Re: [v3] libstdc++/54296

2012-09-05 Thread François Dumont
On 09/05/2012 11:58 AM, Paolo Carlini wrote: Hi, On 09/04/2012 10:08 PM, François Dumont wrote: Hi I managed to do the test with Valgrind and so confirm the fix with the attached patch (unmodified since last proposal). Patch is Ok, thanks for your patience and thanks again for all your g

Re: [v3] libstdc++/54296

2012-08-28 Thread Jonathan Wakely
On 28 August 2012 11:08, François Dumont wrote: > (erase(const key_type&)): Use latters. Let's put "Use the new member functions" here in the ChangeLog, I don't think you can make a plural out of "latter" :-) > * testsuite/23_containers/unordered_map/erase/54296.cc: New. > * testsuit

[v3] libstdc++/54296

2012-08-28 Thread François Dumont
Hi Here is the patch for this issue. I introduced 2 distinct methods to erase elements from a key. The one when keys are unique is rather simple and now use the same underlying code that the erase method from iterator. The other one when keys are not unique first look for nodes matching t