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

Feature gate box patterns #22175

Merged
merged 7 commits into from
Feb 11, 2015
Merged

Conversation

pnkfelix
Copy link
Member

Feature gate box patterns.

Note that this adds a new feature gate, box_patterns specific to e.g. let box i = ..., while leaving box expressions (alone) still guarded via the preexisting box_syntax.

Fix #21931

Switch feature-gate checker from `box_syntax` to `box_patterns` when
visiting a pattern.

(Having to opt into both `box_syntax` and `box_patterns` seemed
unnecessary.)

[breaking-change]
Namely: `collections` (used in `dlist.rs`), `syntax`, `rustc`,
`rustc_typeck`, `rustc_trans`, and `rustdoc`.
@rust-highfive
Copy link
Collaborator

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@nikomatsakis
Copy link
Contributor

r+ but let's change the wording to "experimental" not "experimental in alpha release"

@nikomatsakis nikomatsakis assigned nikomatsakis and unassigned brson Feb 11, 2015
@pnkfelix
Copy link
Member Author

@bors r=nikomatsakis dbe0828

@bors
Copy link
Contributor

bors commented Feb 11, 2015

⌛ Testing commit dbe0828 with merge ff49986...

@bors
Copy link
Contributor

bors commented Feb 11, 2015

💔 Test failed - auto-mac-64-opt

@pnkfelix
Copy link
Member Author

what the heck? That test passed when I make check-stage1 locally

@pnkfelix
Copy link
Member Author

argh! Its because i changed the error message!

@pnkfelix
Copy link
Member Author

@bors r=nikomatsakis cdd8a5a

@bors
Copy link
Contributor

bors commented Feb 11, 2015

⌛ Testing commit cdd8a5a with merge 446bc89...

bors added a commit that referenced this pull request Feb 11, 2015
…akis

Feature gate `box` patterns.

Note that this adds a new feature gate, `box_patterns` specific to e.g. `let box i = ...`, while leaving  `box` expressions (alone) still guarded via the preexisting `box_syntax`.
@bors bors merged commit cdd8a5a into rust-lang:master Feb 11, 2015
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.

Tracking issue for feature-gate box patterns (RFC 469)
5 participants