Skip to content

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

Merged
merged 1 commit into from
Feb 4, 2015
Merged

add naivest entry API to VecMap #21882

merged 1 commit into from
Feb 4, 2015

Conversation

Gankra
Copy link
Contributor

@Gankra Gankra commented Feb 3, 2015

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

@huonw
Copy link
Member

huonw commented Feb 3, 2015

@bors r+ e6ef2da

@bors
Copy link
Collaborator

bors commented Feb 3, 2015

🙀 You have the wrong number! Please try again with 68493ef.

@Gankra
Copy link
Contributor Author

Gankra commented Feb 3, 2015

@bors r=huonw 88449

@@ -13,6 +13,8 @@

#![allow(missing_docs)]

pub use self::Entry::*;
Copy link
Contributor

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?

Copy link
Contributor Author

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.

Copy link
Member

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

Copy link
Contributor Author

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.

@bors
Copy link
Collaborator

bors commented Feb 3, 2015

⌛ Testing commit 88449a8 with merge 7818e90...

@bors
Copy link
Collaborator

bors commented Feb 3, 2015

💔 Test failed - auto-win-64-opt

@alexcrichton
Copy link
Member

@bors: retry

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Feb 4, 2015
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
@bors
Copy link
Collaborator

bors commented Feb 4, 2015

⌛ Testing commit 88449a8 with merge 82c7347...

@alexcrichton alexcrichton reopened this Feb 4, 2015
@alexcrichton
Copy link
Member

@bors: retry

@alexcrichton alexcrichton reopened this Feb 4, 2015
@alexcrichton
Copy link
Member

(sorry about the weird activity, try to kick homu)

@bors bors merged commit 88449a8 into rust-lang:master Feb 4, 2015
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

Successfully merging this pull request may close these issues.

6 participants