diff options
author | Anthony Ramine <n.oxyde@gmail.com> | 2017-02-16 09:18:02 +0100 |
---|---|---|
committer | Anthony Ramine <n.oxyde@gmail.com> | 2017-02-16 18:37:14 +0100 |
commit | 3eed8a91a1183e22b69b1be48ad97a253bc1dc0a (patch) | |
tree | ba6b0b4740560d40445e2a817397a674ec1f8637 /components/script_plugins/ban.rs | |
parent | 84a44a401424852bc44ef5e751e84544ed892e70 (diff) | |
download | servo-3eed8a91a1183e22b69b1be48ad97a253bc1dc0a.tar.gz servo-3eed8a91a1183e22b69b1be48ad97a253bc1dc0a.zip |
Move script lints to script_plugins
The plugins crate now just allows to hook into clippy from a single crate.
Diffstat (limited to 'components/script_plugins/ban.rs')
-rw-r--r-- | components/script_plugins/ban.rs | 53 |
1 files changed, 53 insertions, 0 deletions
diff --git a/components/script_plugins/ban.rs b/components/script_plugins/ban.rs new file mode 100644 index 00000000000..4a4e71c08cc --- /dev/null +++ b/components/script_plugins/ban.rs @@ -0,0 +1,53 @@ +/* This Source Code Form is subject to the terms of the Mozilla Public + * License, v. 2.0. If a copy of the MPL was not distributed with this + * file, You can obtain one at http://mozilla.org/MPL/2.0/. */ + +use rustc::lint::{EarlyContext, LintPass, LintArray, EarlyLintPass, LintContext}; +use syntax::ast::Ty; +use utils::match_ty_unwrap; + +declare_lint!(BANNED_TYPE, Deny, + "Ban various unsafe type combinations"); + +/// Lint for banning various unsafe types +/// +/// Banned types: +/// +/// - `Cell<JSVal>` +/// - `Cell<JS<T>>` +pub struct BanPass; + +impl LintPass for BanPass { + fn get_lints(&self) -> LintArray { + lint_array!(BANNED_TYPE) + } +} + +impl EarlyLintPass for BanPass { + fn check_ty(&mut self, cx: &EarlyContext, ty: &Ty) { + if match_ty_unwrap(ty, &["std", "cell", "Cell"]) + .and_then(|t| t.get(0)) + .and_then(|t| match_ty_unwrap(&**t, &["dom", "bindings", "js", "JS"])) + .is_some() { + cx.span_lint(BANNED_TYPE, ty.span, "Banned type Cell<JS<T>> detected. Use MutJS<JS<T>> instead") + } + if match_ty_unwrap(ty, &["std", "cell", "Cell"]) + .and_then(|t| t.get(0)) + .and_then(|t| match_ty_unwrap(&**t, &["js", "jsval", "JSVal"])) + .is_some() { + cx.span_lint(BANNED_TYPE, ty.span, "Banned type Cell<JSVal> detected. Use MutJS<JSVal> instead") + } + if match_ty_unwrap(ty, &["dom", "bindings", "cell", "DOMRefCell"]) + .and_then(|t| t.get(0)) + .and_then(|t| match_ty_unwrap(&**t, &["dom", "bindings", "js", "JS"])) + .is_some() { + cx.span_lint(BANNED_TYPE, ty.span, "Banned type DOMRefCell<JS<T>> detected. Use MutJS<JS<T>> instead") + } + if match_ty_unwrap(ty, &["dom", "bindings", "cell", "DOMRefCell"]) + .and_then(|t| t.get(0)) + .and_then(|t| match_ty_unwrap(&**t, &["js", "jsapi", "Heap"])) + .is_some() { + cx.span_lint(BANNED_TYPE, ty.span, "Banned type DOMRefCell<Heap<T>> detected. Use Heap<T> directly instead") + } + } +} |