Re: [PATCH] PR libstdc++/91620 Implement DR 526 for std::[forward_]list::remove_if/unique

2020-08-11 Thread Jonathan Wakely via Gcc-patches
On 11/08/20 10:11 +0100, Jonathan Wakely wrote: On 27/12/19 11:57 +0100, François Dumont wrote: Here is the patch to extend DR 526 to forward_list and list remove_if and unique. As the adopted pattern is simpler I also applied it to the remove methods.     PR libstdc++/91620     *

Re: [PATCH] PR libstdc++/91620 Implement DR 526 for std::[forward_]list::remove_if/unique

2020-08-11 Thread Jonathan Wakely via Gcc-patches
On 27/12/19 11:57 +0100, François Dumont wrote: Here is the patch to extend DR 526 to forward_list and list remove_if and unique. As the adopted pattern is simpler I also applied it to the remove methods.     PR libstdc++/91620     * include/bits/forward_list.tcc

Re: [PATCH] PR libstdc++/91620 Implement DR 526 for std::[forward_]list::remove_if/unique

2020-08-10 Thread François Dumont via Gcc-patches
Gentle reminder, this time with tests. I've added one for list::remove cause I think there was none, for forward_list we had remove_freed.cc. I added // { dg-options "-g -O0" } in the new tests otherwise it doesn't fail, that's life with UB. I know that it can pass also with those options.

Re: [PATCH] PR libstdc++/91620 Implement DR 526 for std::[forward_]list::remove_if/unique

2020-03-03 Thread Jonathan Wakely
On 03/03/20 21:32 +, Jonathan Wakely wrote: On 03/03/20 06:42 +0100, François Dumont wrote: Hi     Isn't it something to fix before gcc 10 release ? No, I don't think so. It's not a regression. (And is not experimental C++20 stuff, and is not just changing tests or docs).

Re: [PATCH] PR libstdc++/91620 Implement DR 526 for std::[forward_]list::remove_if/unique

2020-03-03 Thread Jonathan Wakely
On 03/03/20 06:42 +0100, François Dumont wrote: Hi     Isn't it something to fix before gcc 10 release ? No, I don't think so. It's not a regression.

Re: [PATCH] PR libstdc++/91620 Implement DR 526 for std::[forward_]list::remove_if/unique

2020-03-02 Thread François Dumont
Hi     Isn't it something to fix before gcc 10 release ? François On 12/27/19 11:57 AM, François Dumont wrote: Here is the patch to extend DR 526 to forward_list and list remove_if and unique. As the adopted pattern is simpler I also applied it to the remove methods.     PR

[PATCH] PR libstdc++/91620 Implement DR 526 for std::[forward_]list::remove_if/unique

2019-12-27 Thread François Dumont
Here is the patch to extend DR 526 to forward_list and list remove_if and unique. As the adopted pattern is simpler I also applied it to the remove methods.     PR libstdc++/91620     * include/bits/forward_list.tcc (forward_list<>::remove): Collect nodes     to destroy in an intermediate