Skip to content

Commit

Permalink
Remove suggestion to use #[must_root] for functions
Browse files Browse the repository at this point in the history
  • Loading branch information
Manishearth committed Oct 31, 2014
1 parent ba13e44 commit f6e1411
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion components/plugins/lints.rs
Expand Up @@ -112,7 +112,7 @@ impl LintPass for UnrootedPass {
ast::DefaultBlock => {
for arg in decl.inputs.iter() {
lint_unrooted_ty(cx, &*arg.ty,
"Type must be rooted, use #[must_root] on the fn definition to propagate")
"Type must be rooted")
}
}
_ => () // fn is `unsafe`
Expand Down

8 comments on commit f6e1411

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

saw approval from jdm
at Manishearth@f6e1411

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

merging Manishearth/servo/lint-fix = f6e1411 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

Manishearth/servo/lint-fix = f6e1411 merged ok, testing candidate = a7ff7d4

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

No active merge of candidate f6e1411 found, likely manual push to master

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

merging Manishearth/servo/lint-fix = f6e1411 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

Manishearth/servo/lint-fix = f6e1411 merged ok, testing candidate = 2137470

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

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

fast-forwarding master to auto = 2137470

Please sign in to comment.