Skip to content

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
Contributor

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
Contributor

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