There is no reason for this function to return an object,
since it is always used for getting at the name anyhow.
It's used in the inner loop for some popular functions,
so we want to avoid allocating in it.

Every time splice() is called, another temporary object is created.
This version, which uses plain objects as a sort of Hash Bag,
should only produce one temporary object each time it's called.

Basically, it doesn't make sense to generate those things every time
you search. That generates a bunch of stuff for the GC to clean up,
when, if the user wanted to do another search, it would just need
to re-do it again.

Co-authored-by: Guillaume Gomez <[email protected]>

This should have negligible effect on time, but it cuts about 1MiB
off of resident memory usage.

There's no reason for it to be a string, since it's only used for
de-duplicating the results arrays anyhow.

This basically fixes a search bug introduced by earlier changes.