aboutsummaryrefslogtreecommitdiffstats
path: root/components/script_plugins/ban.rs
diff options
context:
space:
mode:
authorAnthony Ramine <n.oxyde@gmail.com>2017-02-16 09:18:02 +0100
committerAnthony Ramine <n.oxyde@gmail.com>2017-02-16 18:37:14 +0100
commit3eed8a91a1183e22b69b1be48ad97a253bc1dc0a (patch)
treeba6b0b4740560d40445e2a817397a674ec1f8637 /components/script_plugins/ban.rs
parent84a44a401424852bc44ef5e751e84544ed892e70 (diff)
downloadservo-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.rs53
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")
+ }
+ }
+}