Skip to content

Perform cfg attribute processing on MultiModifier-generated items #34654

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Jul 6, 2016

Conversation

jseyfried
Copy link
Contributor

@jseyfried
Copy link
Contributor Author

jseyfried commented Jul 5, 2016

The original fix, #34295, only applied to decorators, not modifiers. After making modifiers at least as powerful as decorators in #34253, we removed the decorator expansion code and instead treated decorators as a special case of modifiers in #34446, so the fix no longer applied.

@nrc
Copy link
Member

nrc commented Jul 5, 2016

@bors: r+

@bors
Copy link
Collaborator

bors commented Jul 5, 2016

📌 Commit 71889e4 has been approved by nrc

@jseyfried jseyfried force-pushed the configure_multi_modified branch from 71889e4 to eda848e Compare July 5, 2016 03:32
@jseyfried jseyfried force-pushed the configure_multi_modified branch from eda848e to bf4911b Compare July 5, 2016 05:59
@jseyfried
Copy link
Contributor Author

@bors r=nrc

@bors
Copy link
Collaborator

bors commented Jul 5, 2016

📌 Commit bf4911b has been approved by nrc

@bors
Copy link
Collaborator

bors commented Jul 6, 2016

⌛ Testing commit bf4911b with merge 34c9cdd...

bors added a commit that referenced this pull request Jul 6, 2016
@bors bors merged commit bf4911b into rust-lang:master Jul 6, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants