Add optional support for indexmap Map/Set types#77
Conversation
|
Ah it seems I forgot to test without default-features/std. |
|
I made the
|
|
Ah, finally, the CI is happy :) |
| let a: IndexSet<_> = [0, 1, 2, 3, 4, 5].into_iter().collect(); | ||
| let b: IndexSet<_> = [3, 4, 5, 6, 7, 8].into_iter().collect(); |
There was a problem hiding this comment.
Just did a small tweak here to avoid using the fully-qualified name. (Not sure if you used an LLM to write this -- that's okay but unnecessarily using fully-qualified names is something Claude in particular is prone to do, and something to watch out for.)
There was a problem hiding this comment.
This one is on me, not llm, I have a habit of using fq names occasionally, but in this case that was indeed bogus, thanks!
Adds an optional feature
indexmapfor theIndexMapandIndexSettypes.The implementation is mostly just copy&pasted btreemap/hashmap, but as those used the very handy
map_diff/set_diffmacros, the implementation is quite minimal.