[Issue 10959] std.algorithm.remove is highly bug-prone

[d-bugmail at puremagic.com]

https://issues.dlang.org/show_bug.cgi?id=10959

Andy Valencia <ajv-271-109-6446 at vsta.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ajv-271-109-6446 at vsta.org

--- Comment #16 from Andy Valencia <ajv-271-109-6446 at vsta.org> ---
(In reply to Seb from comment #14)
> A PR to fix the landmines - https://github.com/dlang/phobos/pull/6154
> 
> It's probably too late to rename `remove` :/

I'd suggest deprecating it, and adding removeAt and removeValue

It is certainly, as several people have commented, a landmine.

--