Skip to content
This repository was archived by the owner on Oct 4, 2020. It is now read-only.

Implements keysSet #48

Closed
wants to merge 1 commit into from
Closed

Implements keysSet #48

wants to merge 1 commit into from

Conversation

kritzcreek
Copy link

Fixes #47

@hdgarrood
Copy link
Contributor

It’s a bit odd to have this here as opposed to in maps, I think.

@kritzcreek
Copy link
Author

kritzcreek commented Feb 2, 2018

@hdgarrood sets depends on maps, so it can't be in maps

@hdgarrood
Copy link
Contributor

Right, sorry, I’m aware of that. I mean from the perspective of what API makes sense. I’m saying, perhaps we should consider reorganising things.

@kritzcreek
Copy link
Author

I see, yeah I could see an ordered-containers package which contains all the structures depending on the ordered trie.

@hdgarrood
Copy link
Contributor

See also #46. I think it makes sense to at least put maps and sets together, especially in light of this.

@kritzcreek kritzcreek closed this May 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants