Skip to content

Incoming #2706

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 51 commits into from
Jun 22, 2012
Merged

Incoming #2706

merged 51 commits into from
Jun 22, 2012

Conversation

brson
Copy link
Contributor

@brson brson commented Jun 22, 2012

No description provided.

catamorphism and others added 30 commits June 20, 2012 20:12
The same dtor was getting re-used for different instances, which
didn't always work right. Fixed.
It wasn't possible to refer to the constructor for a class nested inside
an item from the class's outer scope. Fixed.
The fix in 208621 means you now can't shadow a class name with a local,
which is consistent with other behavior. But stackwalk was doing that.
Fixed it.
Irritatingly, class dtors have a different type from resource
dtors (because class dtors have a self argument), and the monomorphic
case wasn't reflecting that. Fixed.
Ports now are represented internally as classes.
Related to issue #2657, but this is not a complete fix.
eholk and others added 21 commits June 21, 2012 16:11
The first benchmark shows about twice the throughput of the old system.
This reverts commit 290206b.

Forgot to add the actual native code, temporarily reverting.
A cleaner way to handle suffix inference for unary minus exprs.
…f field

This was a call to span_bug() before. I'm not sure about the other cases,
but the test case shows that the `vk_self` case can certainly arise with
a bad program, so it should be a span_err() thing and not a span_bug() thing.

Closes #2590
issue-912 is xfailed. issue-2111 is already fixed, but it's good to
have the test case.
brson added a commit that referenced this pull request Jun 22, 2012
@brson brson merged commit 5281771 into master Jun 22, 2012
celinval added a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024
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.

7 participants