slices.Compact

[Deleplace]

Hello, while researching for this Go proposal I noticed this code:

slices.Compact(labels)

which probably doesn't behave as intended and should look like this instead:

labels = slices.Compact(labels)

I have not tried to execute it.

[tlimoncelli]

Interestingly enough... there's no need to compact labels. The items in labels come from map keys, which are by definition unique.

That said... this is an opportunity to clean up some of the code. Thanks for pointing this out!

[Deleplace]

You may consider golang.org/x/exp/maps.Keys, which is handy.

For other interesting reasons, map.Keys is not yet part of the stdlib.