Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a LinkedHashSet #10

Open
apasel422 opened this issue Mar 19, 2015 · 11 comments
Open

Add a LinkedHashSet #10

apasel422 opened this issue Mar 19, 2015 · 11 comments

Comments

@apasel422
Copy link
Contributor

No description provided.

@Gankra
Copy link
Contributor

Gankra commented Mar 19, 2015

I'm not sure I've ever seen something like that

@apasel422
Copy link
Contributor Author

@Gankra
Copy link
Contributor

Gankra commented Mar 19, 2015

Whelp, all right.

@apasel422
Copy link
Contributor Author

Where should this type live? linked_hash_map::LinkedHashSet?

@Gankra
Copy link
Contributor

Gankra commented Jul 12, 2015

It needs a seperate module for iterators.

@apasel422
Copy link
Contributor Author

linked_hash_map::set::LinkedHashSet, then? Or a separate crate? If the former, should there be a top-level re-export?

@Gankra
Copy link
Contributor

Gankra commented Jul 12, 2015

I don't have any strong feelings on the matter.

On Sun, Jul 12, 2015 at 3:52 PM, Andrew Paseltiner <notifications@github.com

wrote:

linked_hash_map::set::LinkedHashSet, then? Or a separate crate? If the
former, should there be a top-level re-export?


Reply to this email directly or view it on GitHub
#10 (comment)
.

@nugend
Copy link

nugend commented Oct 12, 2016

Found myself looking for this. Trying to do cycle detection and report on the observed cycle.

@apasel422
Copy link
Contributor Author

@nugend For the time being, you can use LinkedHashMap<SomeKey, ()>.

@nugend
Copy link

nugend commented Oct 12, 2016

Ah, good idea!

@alexheretic
Copy link

alexheretic commented Oct 25, 2017

I just published linked_hash_set, based on this project for anyone looking for LinkedHashSet.

Maintainers: If linked-hash-map ever spins up again feel free to grab the set code as you please. I think it makes sense to have along side the actual map implementation. If that ever happens lets me know so I can act accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants