mirror of
https://github.com/servo/servo.git
synced 2025-08-03 04:30:10 +01:00
Auto merge of #10804 - tylersouthwick:remove-plugins-utils-unsafe_context, r=nox
Remove unused code and make utils private fixes #10771 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10804) <!-- Reviewable:end -->
This commit is contained in:
commit
2b68cf4aba
2 changed files with 1 additions and 25 deletions
|
@ -39,7 +39,7 @@ pub mod lints;
|
||||||
/// Autogenerates implementations of Reflectable on DOM structs
|
/// Autogenerates implementations of Reflectable on DOM structs
|
||||||
pub mod reflector;
|
pub mod reflector;
|
||||||
/// Utilities for writing plugins
|
/// Utilities for writing plugins
|
||||||
pub mod utils;
|
mod utils;
|
||||||
|
|
||||||
#[plugin_registrar]
|
#[plugin_registrar]
|
||||||
pub fn plugin_registrar(reg: &mut Registry) {
|
pub fn plugin_registrar(reg: &mut Registry) {
|
||||||
|
|
|
@ -73,30 +73,6 @@ pub fn match_lang_did(cx: &LateContext, did: DefId, value: &str) -> bool {
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
||||||
// Determines if a block is in an unsafe context so that an unhelpful
|
|
||||||
// lint can be aborted.
|
|
||||||
pub fn unsafe_context(map: &ast_map::Map, id: ast::NodeId) -> bool {
|
|
||||||
match map.find(map.get_parent(id)) {
|
|
||||||
Some(ast_map::NodeImplItem(itm)) => {
|
|
||||||
match itm.node {
|
|
||||||
hir::ImplItemKind::Method(ref sig, _) => sig.unsafety == hir::Unsafety::Unsafe,
|
|
||||||
_ => false
|
|
||||||
}
|
|
||||||
},
|
|
||||||
Some(ast_map::NodeItem(itm)) => {
|
|
||||||
match itm.node {
|
|
||||||
hir::ItemFn(_, style, _, _, _, _) => match style {
|
|
||||||
hir::Unsafety::Unsafe => true,
|
|
||||||
_ => false,
|
|
||||||
},
|
|
||||||
_ => false,
|
|
||||||
}
|
|
||||||
}
|
|
||||||
_ => false // There are probably a couple of other unsafe cases we don't care to lint, those will need
|
|
||||||
// to be added.
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/// check if a DefId's path matches the given absolute type path
|
/// check if a DefId's path matches the given absolute type path
|
||||||
/// usage e.g. with
|
/// usage e.g. with
|
||||||
/// `match_def_path(cx, id, &["core", "option", "Option"])`
|
/// `match_def_path(cx, id, &["core", "option", "Option"])`
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue