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

Visit statement and expression attributes in the AST visitor #34199

Merged
merged 2 commits into from
Jun 10, 2016

Conversation

jseyfried
Copy link
Contributor

@jseyfried jseyfried commented Jun 10, 2016

Currently, these attributes are not visited, so they are not gated feature checked in the post expansion visitor. This only affects crates using #![feature(stmt_expr_attributes)].
r? @nrc

@durka
Copy link
Contributor

durka commented Jun 10, 2016

Does this move these attributes early enough to support #[allow(...)] on stmts/exprs?

@nrc
Copy link
Member

nrc commented Jun 10, 2016

@bors: r+

@bors
Copy link
Contributor

bors commented Jun 10, 2016

📌 Commit 8475a4b has been approved by nrc

@bors
Copy link
Contributor

bors commented Jun 10, 2016

⌛ Testing commit 8475a4b with merge 8c6bd23...

bors added a commit that referenced this pull request Jun 10, 2016
Visit statement and expression attributes in the AST visitor

Currently, these attributes are not visited, so they are not gated feature checked in the post expansion visitor. This only affects crates using `#![feature(stmt_expr_attributes)]`.
r? @nrc
@bors
Copy link
Contributor

bors commented Jun 10, 2016

💔 Test failed - auto-mac-32-opt

@jseyfried
Copy link
Contributor Author

@bors retry force

@jseyfried
Copy link
Contributor Author

jseyfried commented Jun 10, 2016

@durka

Does this move these attributes early enough to support #[allow(...)] on stmts/exprs?

No -- it looks like #[allow(...)] still has no effect on stmts/exprs. Is there an issue for this?

@bors bors merged commit 8475a4b into rust-lang:master Jun 10, 2016
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.

4 participants