-
Notifications
You must be signed in to change notification settings - Fork 13.3k
add naivest entry API to VecMap #21882
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
Conversation
@bors r+ e6ef2da |
🙀 You have the wrong number! Please try again with |
@bors r=huonw 88449 |
@@ -13,6 +13,8 @@ | |||
|
|||
#![allow(missing_docs)] | |||
|
|||
pub use self::Entry::*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you export this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I basically was copy-pasting from BTreeMap, I didn't even notice the pub
.
I'm going to refactor this all in my next PR anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not get rid of it now? One fewer thing to remember
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't be home to fix this for a while.
⌛ Testing commit 88449a8 with merge 7818e90... |
💔 Test failed - auto-win-64-opt |
@bors: retry |
Building over night, posting for review now. Presumably not much should need change. I consider this necessary to move forward with a proper stabilization of the API. r? @huonw
⌛ Testing commit 88449a8 with merge 82c7347... |
@bors: retry |
(sorry about the weird activity, try to kick homu) |
Building over night, posting for review now. Presumably not much should need change.
I consider this necessary to move forward with a proper stabilization of the API.
r? @huonw