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

syntax/rustc_front: Simplify VariantData::fields #29303

Merged
merged 1 commit into from
Oct 26, 2015

Conversation

petrochenkov
Copy link
Contributor

And use VariantData instead of P<VariantData> in Item_ and Variant_

Improvements suggested by @eddyb in #28816 (comment) and #28816 (comment)

plugin-[breaking-change]

r? @eddyb

And use VariantData instead of P<VariantData> in Item_ and Variant_
@eddyb
Copy link
Member

eddyb commented Oct 25, 2015

@bors r+

@bors
Copy link
Contributor

bors commented Oct 25, 2015

📌 Commit e8ddbba has been approved by eddyb

bors added a commit that referenced this pull request Oct 26, 2015
And use `VariantData` instead of `P<VariantData>` in `Item_` and `Variant_`

Improvements suggested by @eddyb in #28816 (comment) and #28816 (comment)

plugin-[breaking-change]

r? @eddyb
@bors
Copy link
Contributor

bors commented Oct 26, 2015

⌛ Testing commit e8ddbba with merge c44316d...

@bors bors merged commit e8ddbba into rust-lang:master Oct 26, 2015
@petrochenkov petrochenkov deleted the unistrimp branch November 22, 2015 11:34
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.

3 participants