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.

Sign up for GitHub

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

Jump to bottom

Grab bag #302

Closed
wants to merge 3 commits into from
Closed

Grab bag #302

wants to merge 3 commits into from

Conversation

brson
Copy link
Contributor

@brson brson commented Mar 31, 2011

  • Removes some expression parsing that is done in multiple places
  • Reverts an incorrect change I made to expr_block translation
  • incr_all_refcnts -> take_ty

@brson
Remove redundant parser logic.
50aae2a 
These cases are all dealt with in parse_expr_inner.
@brson
Restore scope block contexts to the translation of expr_block.
1768989 
I removed this previously but that was wrong because it caused
block expressions to not create a new scope.
@brson
Rename incr_all_refcnts to take_ty, to match drop_ty
11289f8
@graydon
Copy link
Contributor

graydon commented Mar 31, 2011

Integrated, thanks.

@graydon graydon closed this Mar 31, 2011
oli-obk pushed a commit to oli-obk/rust that referenced this pull request Sep 19, 2017
@RalfJung
Merge pull request rust-lang#302 from RalfJung/mir-validate2
5d60c61 
enable a test that waited for a rustc fix
ZuseZ4 pushed a commit to EnzymeAD/rust that referenced this pull request Mar 7, 2023
@tgymnich
replace malloc with new (rust-lang#302)
820a0eb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brson @graydon