diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2017-02-16 10:58:19 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-02-16 10:58:19 -0800 |
commit | a099d27f99dadf3f7c26d997e43c1a12e0c5bd0e (patch) | |
tree | 5583d83e8d579d1d7f93887eb7f8a54442161e7a /components/script_plugins/ban.rs | |
parent | 05623b36a15b594bbc690fcd8e3b642995618de1 (diff) | |
parent | 3eed8a91a1183e22b69b1be48ad97a253bc1dc0a (diff) | |
download | servo-a099d27f99dadf3f7c26d997e43c1a12e0c5bd0e.tar.gz servo-a099d27f99dadf3f7c26d997e43c1a12e0c5bd0e.zip |
Auto merge of #15584 - nox:plugin, r=jdm
Move script lints to script_plugins
<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/15584)
<!-- Reviewable:end -->
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") + } + } +} |