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

Eschew arena::Arena in variance.rs #21987

Merged
merged 1 commit into from
Feb 7, 2015

Conversation

pnkfelix
Copy link
Member

@pnkfelix pnkfelix commented Feb 5, 2015

Use a TypedArena rather than an Arena for the backing storage of variance analysis.

(The code was not using the main feature of Arena in the first place, since all of the alloc calls were on the same type.)

…ance analysis.

(The code was not using the main feature of Arena in the first place,
since all of the `alloc` calls were on the same type.
@rust-highfive
Copy link
Collaborator

r? @huonw

(rust_highfive has picked a reviewer for you, use r? to override)

@pnkfelix
Copy link
Member Author

pnkfelix commented Feb 5, 2015

spawned off of #21972.

@huonw
Copy link
Member

huonw commented Feb 5, 2015

@bors r+ 50c674 rollup

@pnkfelix
Copy link
Member Author

pnkfelix commented Feb 5, 2015

(also, anyone can review this. It is a trivial refactoring from one API to another.)

@reem
Copy link
Contributor

reem commented Feb 5, 2015

@huonw you dropped a 2 at the end of that hash, does bors still catch that?

@pnkfelix
Copy link
Member Author

pnkfelix commented Feb 5, 2015

@bors r=huonw 50c6742 rollup

@pnkfelix
Copy link
Member Author

pnkfelix commented Feb 5, 2015

@reem (apparently not!)

@huonw
Copy link
Member

huonw commented Feb 5, 2015

@reem it catches any prefix that's at least 4 characters long (at least, I think that was the behaviour last time I talked to @barosl), in any case, @bors is proactive about alerting about malformed r+'s.

@bors
Copy link
Contributor

bors commented Feb 5, 2015

🙀 You have the wrong number! Please try again with 50c6742.

@huonw
Copy link
Member

huonw commented Feb 5, 2015

(Like that, I guess.)

Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 6, 2015
…nalysis, r=huonw

 Use a TypedArena rather than an Arena for the backing storage of variance analysis.

(The code was not using the main feature of Arena in the first place, since all of the `alloc` calls were on the same type.)
@bors bors merged commit 50c6742 into rust-lang:master Feb 7, 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.

5 participants