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

(blocked on #5121) add stronger region annotations to the maps #5670

Closed
wants to merge 1 commit into from
Closed

Conversation

thestinger
Copy link
Contributor

No description provided.

@catamorphism
Copy link
Contributor

@thestinger Sorry, but can you re-rebase?

@thestinger
Copy link
Contributor Author

I'm having trouble getting the core-map.rs test to compile right now, I think it might be a bug.

/home/strcat/projects/rust/src/test/bench/core-map.rs:26:22: 26:23 error: Illegal lifetime 'r: only 'static is allowed here
/home/strcat/projects/rust/src/test/bench/core-map.rs:26 fn ascending<M: Map<'r, uint, uint>>(map: &mut M, n_keys: uint) {
                                                                               ^
/home/strcat/projects/rust/src/test/bench/core-map.rs:48:23: 48:24 error: Illegal lifetime 'r: only 'static is allowed here
/home/strcat/projects/rust/src/test/bench/core-map.rs:48 fn descending<M: Map<'r, uint, uint>>(map: &mut M, n_keys: uint) {
                                                                                ^
/home/strcat/projects/rust/src/test/bench/core-map.rs:70:19: 70:20 error: Illegal lifetime 'r: only 'static is allowed here
/home/strcat/projects/rust/src/test/bench/core-map.rs:70 fn vector<M: Map<'r, uint, uint>>(map: &mut M, n_keys: uint, dist: &[uint]) {

@nikomatsakis: Am I doing something wrong there?

@nikomatsakis
Copy link
Contributor

We mentioned it on IRC, but for the record this is #5121

@brson
Copy link
Contributor

brson commented May 10, 2013

@thestinger What should we do to move this forward?

@thestinger
Copy link
Contributor Author

@brson: this is still blocked on #5121 but I'll just reopen a new one when it's fixed. This is too out-of-date now.

@thestinger thestinger closed this May 10, 2013
calebcartwright added a commit to calebcartwright/rust that referenced this pull request Jan 24, 2023
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.

4 participants