Skip to content

Add support for registering attributes with rustc in plugins #25168

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

Merged
merged 4 commits into from
May 24, 2015

Conversation

Manishearth
Copy link
Member

This lets plugin authors opt attributes out of the custom_attribute
and unused_attribute checks.

cc @thepowersgang

This lets plugin authors opt attributes out of the `custom_attribute`
and `unused_attribute` checks.
@Manishearth
Copy link
Member Author

I added a test

r? @huonw


/// Register an attribute with an attribute type
///
/// Registered attributes will bypass the `custom_attribute` feature gate
Copy link
Member

Choose a reason for hiding this comment

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

This and the next two lines can be one paragraph. Also, all these sentences need .s.

@Manishearth
Copy link
Member Author

(addressed)

@@ -130,4 +135,19 @@ impl<'a> Registry<'a> {
pub fn register_llvm_pass(&mut self, name: &str) {
self.llvm_passes.push(name.to_owned());
}


/// Register an attribute with an attribute type
Copy link
Member

Choose a reason for hiding this comment

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

Needs a .

@Manishearth
Copy link
Member Author

(addressed)

@eddyb
Copy link
Member

eddyb commented May 24, 2015

@bors r+

@bors
Copy link
Collaborator

bors commented May 24, 2015

📌 Commit 22b720a has been approved by eddyb

@bors
Copy link
Collaborator

bors commented May 24, 2015

⌛ Testing commit 22b720a with merge 3738537...

bors added a commit that referenced this pull request May 24, 2015
This lets plugin authors opt attributes out of the `custom_attribute`
and `unused_attribute` checks.


cc @thepowersgang
@bors
Copy link
Collaborator

bors commented May 24, 2015

💔 Test failed - auto-mac-32-opt

@Manishearth
Copy link
Member Author

@bors: retry

bors added a commit that referenced this pull request May 24, 2015
This lets plugin authors opt attributes out of the `custom_attribute`
and `unused_attribute` checks.


cc @thepowersgang
@bors
Copy link
Collaborator

bors commented May 24, 2015

⌛ Testing commit 22b720a with merge cc56c20...

@bors bors merged commit 22b720a into rust-lang:master May 24, 2015
@Manishearth Manishearth deleted the register_attr branch May 24, 2015 11:22
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