Skip to content

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

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants