Skip to content
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

Handle gateage of built-in attributes seperately #28077

Merged
merged 1 commit into from
Aug 29, 2015
Merged

Handle gateage of built-in attributes seperately #28077

merged 1 commit into from
Aug 29, 2015

Conversation

jonas-schievink
Copy link
Contributor

This allows marking attributes as whitelisted/crate-only independent of
their feature gate status.

Closes #24213

@rust-highfive
Copy link
Collaborator

r? @huonw

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

This allows marking attributes as whitelisted/crate-only independent of
their feature gate status.

Closes #24213
("link_section", Whitelisted, Ungated),
("no_builtins", Whitelisted, Ungated),
("no_mangle", Whitelisted, Ungated),
("no_debug", Whitelisted, Ungated),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, not related to the patch, but I wonder if some of these debug things should be gated... (Edit: opened #28091.)

@huonw
Copy link
Member

huonw commented Aug 29, 2015

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented Aug 29, 2015

📌 Commit 4b571b0 has been approved by huonw

@bors
Copy link
Contributor

bors commented Aug 29, 2015

⌛ Testing commit 4b571b0 with merge 9982314...

bors added a commit that referenced this pull request Aug 29, 2015
This allows marking attributes as whitelisted/crate-only independent of
their feature gate status.

Closes #24213
@bors bors merged commit 4b571b0 into rust-lang:master Aug 29, 2015
@jonas-schievink jonas-schievink deleted the attr-gateage branch August 30, 2015 00:16
# 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.

4 participants