Skip to content

FromIterator implementations missing #7626

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.

Sign up for GitHub

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

Jump to bottom
Closed
hanny24 opened this issue Jul 6, 2013 · 3 comments
Closed

FromIterator implementations missing #7626

hanny24 opened this issue Jul 6, 2013 · 3 comments

Comments

@hanny24
Copy link
Contributor

hanny24 commented Jul 6, 2013

FromIterator is currently implemented only by vector, however it should be implemented by almost all containers (hashmap, triemap, treemap, ...)
@hanny24
Copy link
Contributor Author

hanny24 commented Jul 6, 2013

I'm willing to work on this

@huonw huonw mentioned this issue Jul 14, 2013
Closed
@MarkJr94
Copy link
Contributor

I'm going to try and work on it. starting with std::hashmap.

@MarkJr94 MarkJr94 mentioned this issue Jul 14, 2013
Closed
bors added a commit that referenced this issue Jul 14, 2013
@bors
auto merge of #7788 : MarkJr94/rust/from_iter, r=cmr
0cb1ac0 
Added Iterators for HashMap/Set, TreeMap/Set, TrieMap/Set, and PriorityQueue as per Issue #7626
@Kimundi
Copy link
Member

Kimundi commented Jul 29, 2013

This can be closed now.

flip1995 pushed a commit to flip1995/rust that referenced this issue Sep 8, 2021
@bors
Auto merge of rust-lang#7627 - xFrednet:0000-updating-issue-templates…
df7e63b 
…-again, r=llogiq

Updating issue templates again for rustbot

It turns out that our current issue template can sometimes trigger a rustbot error message, as can be seen in [rust-lang#7626](rust-lang/rust-clippy#7626). I originally tested this in rust-lang#7599, but it's apparently a bit inconsistent. This PR adds backticks to the commands, as correctly suggested by `@mikerite` in the comments. (Thank you!)

``@rustbot` label +S-blocked`

---

Now I also pushed a tiny link fix as well. 🙃

---

changelog: none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@MarkJr94 @thestinger @Kimundi @hanny24