Re: [elixir-core:10447] [Proposal] Map.filter/2 based on :maps.filter

2021-09-11 Thread Wiebe-Marten Wijnja
@Allen Madsen: I have added `Enum.filter` to the benchmark. Also, by telling the compiler to inline the calls to `next` and `iterator`, removing redundant guard-clause checks and not supporting map-iterators (as the rest of the `Map`-module doesn't expose them either), `Map.filter` is able to be

Re: [elixir-core:10446] [Proposal] Map.filter/2 based on :maps.filter

2021-09-11 Thread José Valim
Thanks Marten, so I think we can proceed with a PR for Map.filter and Map.map, both passing tuples as argument. Note we will need the same for Keyword. Could you please submit it? On Fri, Sep 10, 2021 at 11:08 PM Allen Madsen wrote: > Perhaps also benchmark the combination of Enum.filter/2 and