Skip to content

Commit 65c103d

Browse files
oli-obkManishearth
oli-obk
authored and
Manishearth
committed
WIP: use symbols instead of strings
1 parent 42480fd commit 65c103d

File tree

138 files changed

+1265
-694
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

138 files changed

+1265
-694
lines changed

‎clippy_lints/src/approx_const.rs

+19-17
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,12 @@
11
use crate::utils::span_lint;
2+
use crate::utils::sym;
23
use rustc::hir::*;
34
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
45
use rustc::{declare_lint_pass, declare_tool_lint};
56
use std::f64::consts as f64;
67
use syntax::ast::{FloatTy, Lit, LitKind};
78
use syntax::symbol;
9+
use syntax::symbol::Symbol;
810

911
declare_clippy_lint! {
1012
/// **What it does:** Checks for floating point literals that approximate
@@ -31,23 +33,23 @@ declare_clippy_lint! {
3133
}
3234

3335
// Tuples are of the form (constant, name, min_digits)
34-
const KNOWN_CONSTS: &[(f64, &str, usize)] = &[
35-
(f64::E, "E", 4),
36-
(f64::FRAC_1_PI, "FRAC_1_PI", 4),
37-
(f64::FRAC_1_SQRT_2, "FRAC_1_SQRT_2", 5),
38-
(f64::FRAC_2_PI, "FRAC_2_PI", 5),
39-
(f64::FRAC_2_SQRT_PI, "FRAC_2_SQRT_PI", 5),
40-
(f64::FRAC_PI_2, "FRAC_PI_2", 5),
41-
(f64::FRAC_PI_3, "FRAC_PI_3", 5),
42-
(f64::FRAC_PI_4, "FRAC_PI_4", 5),
43-
(f64::FRAC_PI_6, "FRAC_PI_6", 5),
44-
(f64::FRAC_PI_8, "FRAC_PI_8", 5),
45-
(f64::LN_10, "LN_10", 5),
46-
(f64::LN_2, "LN_2", 5),
47-
(f64::LOG10_E, "LOG10_E", 5),
48-
(f64::LOG2_E, "LOG2_E", 5),
49-
(f64::PI, "PI", 3),
50-
(f64::SQRT_2, "SQRT_2", 5),
36+
const KNOWN_CONSTS: &[(f64, Symbol, usize)] = &[
37+
(f64::E, *sym::E, 4),
38+
(f64::FRAC_1_PI, *sym::FRAC_1_PI, 4),
39+
(f64::FRAC_1_SQRT_2, *sym::FRAC_1_SQRT_2, 5),
40+
(f64::FRAC_2_PI, *sym::FRAC_2_PI, 5),
41+
(f64::FRAC_2_SQRT_PI, *sym::FRAC_2_SQRT_PI, 5),
42+
(f64::FRAC_PI_2, *sym::FRAC_PI_2, 5),
43+
(f64::FRAC_PI_3, *sym::FRAC_PI_3, 5),
44+
(f64::FRAC_PI_4, *sym::FRAC_PI_4, 5),
45+
(f64::FRAC_PI_6, *sym::FRAC_PI_6, 5),
46+
(f64::FRAC_PI_8, *sym::FRAC_PI_8, 5),
47+
(f64::LN_10, *sym::LN_10, 5),
48+
(f64::LN_2, *sym::LN_2, 5),
49+
(f64::LOG10_E, *sym::LOG10_E, 5),
50+
(f64::LOG2_E, *sym::LOG2_E, 5),
51+
(f64::PI, *sym::PI, 3),
52+
(f64::SQRT_2, *sym::SQRT_2, 5),
5153
];
5254

5355
declare_lint_pass!(ApproxConstant => [APPROX_CONSTANT]);

‎clippy_lints/src/arithmetic.rs

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
use crate::consts::constant_simple;
22
use crate::utils::span_lint;
3+
use crate::utils::sym;
34
use rustc::hir;
45
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
56
use rustc::{declare_tool_lint, impl_lint_pass};

‎clippy_lints/src/assertions_on_constants.rs

+3-2
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ use syntax_pos::Span;
66

77
use crate::consts::{constant, Constant};
88
use crate::utils::{in_macro_or_desugar, is_direct_expn_of, span_help_and_lint};
9+
use crate::utils::sym;
910

1011
declare_clippy_lint! {
1112
/// **What it does:** Checks for `assert!(true)` and `assert!(false)` calls.
@@ -40,9 +41,9 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for AssertionsOnConstants {
4041
!in_macro_or_desugar(span)
4142
};
4243
if_chain! {
43-
if let Some(assert_span) = is_direct_expn_of(e.span, "assert");
44+
if let Some(assert_span) = is_direct_expn_of(e.span, *sym::assert);
4445
if !in_macro_or_desugar(assert_span)
45-
|| is_direct_expn_of(assert_span, "debug_assert")
46+
|| is_direct_expn_of(assert_span, *sym::debug_assert)
4647
.map_or(false, debug_assert_not_in_macro_or_desugar);
4748
if let ExprKind::Unary(_, ref lit) = e.node;
4849
if let Some(bool_const) = constant(cx, cx.tables, lit);

‎clippy_lints/src/assign_ops.rs

+7-2
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@ use crate::utils::{
99
get_trait_def_id, implements_trait, snippet_opt, span_lint_and_then, trait_ref_of_method, SpanlessEq,
1010
};
1111
use crate::utils::{higher, sugg};
12+
use crate::utils::sym;
13+
use syntax::symbol::Symbol;
1214

1315
declare_clippy_lint! {
1416
/// **What it does:** Checks for `a = a op b` or `a = b commutative_op a`
@@ -88,8 +90,11 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for AssignOps {
8890
$($trait_name:ident),+) => {
8991
match $op {
9092
$(hir::BinOpKind::$trait_name => {
91-
let [krate, module] = crate::utils::paths::OPS_MODULE;
92-
let path = [krate, module, concat!(stringify!($trait_name), "Assign")];
93+
let [krate, module] = *crate::utils::paths::OPS_MODULE;
94+
let ident = {
95+
*crate::utils::sym::assign::$trait_name
96+
};
97+
let path: [Symbol; 3] = [krate, module, ident];
9398
let trait_id = if let Some(trait_id) = get_trait_def_id($cx, &path) {
9499
trait_id
95100
} else {

‎clippy_lints/src/attrs.rs

+17-15
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,9 @@
33
use crate::reexport::*;
44
use crate::utils::{
55
in_macro_or_desugar, is_present_in_source, last_line_of_span, paths, snippet_opt, span_lint, span_lint_and_sugg,
6-
span_lint_and_then, without_block_comments,
6+
span_lint_and_then, without_block_comments, match_def_path
77
};
8+
use crate::utils::sym;
89
use if_chain::if_chain;
910
use rustc::hir::*;
1011
use rustc::lint::{
@@ -17,6 +18,7 @@ use rustc_errors::Applicability;
1718
use semver::Version;
1819
use syntax::ast::{AttrStyle, Attribute, Lit, LitKind, MetaItemKind, NestedMetaItem};
1920
use syntax::source_map::Span;
21+
use syntax::symbol::Symbol;
2022

2123
declare_clippy_lint! {
2224
/// **What it does:** Checks for items annotated with `#[inline(always)]`,
@@ -205,14 +207,14 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for Attributes {
205207
},
206208
_ => {},
207209
}
208-
if items.is_empty() || !attr.check_name("deprecated") {
210+
if items.is_empty() || !attr.check_name(*sym::deprecated) {
209211
return;
210212
}
211213
for item in items {
212214
if_chain! {
213215
if let NestedMetaItem::MetaItem(mi) = &item;
214216
if let MetaItemKind::NameValue(lit) = &mi.node;
215-
if mi.check_name("since");
217+
if mi.check_name(*sym::since);
216218
then {
217219
check_semver(cx, item.span(), lit);
218220
}
@@ -228,7 +230,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for Attributes {
228230
}
229231
match item.node {
230232
ItemKind::ExternCrate(..) | ItemKind::Use(..) => {
231-
let skip_unused_imports = item.attrs.iter().any(|attr| attr.check_name("macro_use"));
233+
let skip_unused_imports = item.attrs.iter().any(|attr| attr.check_name(*sym::macro_use));
232234

233235
for attr in &item.attrs {
234236
if in_external_macro(cx.sess(), attr.span) {
@@ -243,15 +245,15 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for Attributes {
243245
for lint in lint_list {
244246
match item.node {
245247
ItemKind::Use(..) => {
246-
if is_word(lint, "unused_imports") || is_word(lint, "deprecated") {
248+
if is_word(lint, *sym::unused_imports) || is_word(lint, *sym::deprecated) {
247249
return;
248250
}
249251
},
250252
ItemKind::ExternCrate(..) => {
251-
if is_word(lint, "unused_imports") && skip_unused_imports {
253+
if is_word(lint, *sym::unused_imports) && skip_unused_imports {
252254
return;
253255
}
254-
if is_word(lint, "unused_extern_crates") {
256+
if is_word(lint, *sym::unused_extern_crates) {
255257
return;
256258
}
257259
},
@@ -395,7 +397,7 @@ fn is_relevant_expr(cx: &LateContext<'_, '_>, tables: &ty::TypeckTables<'_>, exp
395397
ExprKind::Call(path_expr, _) => {
396398
if let ExprKind::Path(qpath) = &path_expr.node {
397399
if let Some(fun_id) = tables.qpath_res(qpath, path_expr.hir_id).opt_def_id() {
398-
!cx.match_def_path(fun_id, &paths::BEGIN_PANIC)
400+
!match_def_path(cx, fun_id, &*paths::BEGIN_PANIC)
399401
} else {
400402
true
401403
}
@@ -441,10 +443,10 @@ fn check_attrs(cx: &LateContext<'_, '_>, span: Span, name: Name, attrs: &[Attrib
441443
}
442444

443445
if let Some(values) = attr.meta_item_list() {
444-
if values.len() != 1 || !attr.check_name("inline") {
446+
if values.len() != 1 || !attr.check_name(*sym::inline) {
445447
continue;
446448
}
447-
if is_word(&values[0], "always") {
449+
if is_word(&values[0], *sym::always) {
448450
span_lint(
449451
cx,
450452
INLINE_ALWAYS,
@@ -473,7 +475,7 @@ fn check_semver(cx: &LateContext<'_, '_>, span: Span, lit: &Lit) {
473475
);
474476
}
475477

476-
fn is_word(nmi: &NestedMetaItem, expected: &str) -> bool {
478+
fn is_word(nmi: &NestedMetaItem, expected: Symbol) -> bool {
477479
if let NestedMetaItem::MetaItem(mi) = &nmi {
478480
mi.is_word() && mi.check_name(expected)
479481
} else {
@@ -487,16 +489,16 @@ impl EarlyLintPass for DeprecatedCfgAttribute {
487489
fn check_attribute(&mut self, cx: &EarlyContext<'_>, attr: &Attribute) {
488490
if_chain! {
489491
// check cfg_attr
490-
if attr.check_name("cfg_attr");
492+
if attr.check_name(*sym::cfg_attr);
491493
if let Some(items) = attr.meta_item_list();
492494
if items.len() == 2;
493495
// check for `rustfmt`
494496
if let Some(feature_item) = items[0].meta_item();
495-
if feature_item.check_name("rustfmt");
497+
if feature_item.check_name(*sym::rustfmt);
496498
// check for `rustfmt_skip` and `rustfmt::skip`
497499
if let Some(skip_item) = &items[1].meta_item();
498-
if skip_item.check_name("rustfmt_skip") ||
499-
skip_item.path.segments.last().expect("empty path in attribute").ident.name == "skip";
500+
if skip_item.check_name(*sym::rustfmt_skip) ||
501+
skip_item.path.segments.last().expect("empty path in attribute").ident.name == *sym::skip;
500502
// Only lint outer attributes, because custom inner attributes are unstable
501503
// Tracking issue: https://github.com/rust-lang/rust/issues/54726
502504
if let AttrStyle::Outer = attr.style;

‎clippy_lints/src/bit_mask.rs

+1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
use crate::consts::{constant, Constant};
22
use crate::utils::sugg::Sugg;
33
use crate::utils::{span_lint, span_lint_and_then};
4+
use crate::utils::sym;
45
use if_chain::if_chain;
56
use rustc::hir::*;
67
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};

‎clippy_lints/src/blacklisted_name.rs

+1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
use crate::utils::span_lint;
2+
use crate::utils::sym;
23
use rustc::hir::*;
34
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
45
use rustc::{declare_tool_lint, impl_lint_pass};

‎clippy_lints/src/booleans.rs

+7-5
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ use crate::utils::{
22
get_trait_def_id, implements_trait, in_macro, in_macro_or_desugar, match_type, paths, snippet_opt,
33
span_lint_and_then, SpanlessEq,
44
};
5+
use crate::utils::sym;
56
use rustc::hir::intravisit::*;
67
use rustc::hir::*;
78
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
@@ -10,6 +11,7 @@ use rustc_data_structures::thin_vec::ThinVec;
1011
use rustc_errors::Applicability;
1112
use syntax::ast::LitKind;
1213
use syntax::source_map::{dummy_spanned, Span, DUMMY_SP};
14+
use syntax::symbol::Symbol;
1315

1416
declare_clippy_lint! {
1517
/// **What it does:** Checks for boolean expressions that can be written more
@@ -50,7 +52,7 @@ declare_clippy_lint! {
5052
}
5153

5254
// For each pairs, both orders are considered.
53-
const METHODS_WITH_NEGATION: [(&str, &str); 2] = [("is_some", "is_none"), ("is_err", "is_ok")];
55+
const METHODS_WITH_NEGATION: [(Symbol, Symbol); 2] = [(*sym::is_some, *sym::is_none), (*sym::is_err, *sym::is_ok)];
5456

5557
declare_lint_pass!(NonminimalBool => [NONMINIMAL_BOOL, LOGIC_BUG]);
5658

@@ -187,16 +189,16 @@ impl<'a, 'tcx, 'v> SuggestContext<'a, 'tcx, 'v> {
187189
},
188190
ExprKind::MethodCall(path, _, args) if args.len() == 1 => {
189191
let type_of_receiver = self.cx.tables.expr_ty(&args[0]);
190-
if !match_type(self.cx, type_of_receiver, &paths::OPTION)
191-
&& !match_type(self.cx, type_of_receiver, &paths::RESULT)
192+
if !match_type(self.cx, type_of_receiver, &*paths::OPTION)
193+
&& !match_type(self.cx, type_of_receiver, &*paths::RESULT)
192194
{
193195
return None;
194196
}
195197
METHODS_WITH_NEGATION
196198
.iter()
197199
.cloned()
198200
.flat_map(|(a, b)| vec![(a, b), (b, a)])
199-
.find(|&(a, _)| a == path.ident.as_str())
201+
.find(|&(a, _)| a == path.ident.name)
200202
.and_then(|(_, neg_method)| Some(format!("{}.{}()", self.snip(&args[0])?, neg_method)))
201203
},
202204
_ => None,
@@ -466,5 +468,5 @@ impl<'a, 'tcx> Visitor<'tcx> for NonminimalBoolVisitor<'a, 'tcx> {
466468

467469
fn implements_ord<'a, 'tcx>(cx: &'a LateContext<'a, 'tcx>, expr: &Expr) -> bool {
468470
let ty = cx.tables.expr_ty(expr);
469-
get_trait_def_id(cx, &paths::ORD).map_or(false, |id| implements_trait(cx, ty, id, &[]))
471+
get_trait_def_id(cx, &*paths::ORD).map_or(false, |id| implements_trait(cx, ty, id, &[]))
470472
}

‎clippy_lints/src/bytecount.rs

+5-4
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ use crate::utils::{
22
contains_name, get_pat_name, match_type, paths, single_segment_path, snippet_with_applicability,
33
span_lint_and_sugg, walk_ptrs_ty,
44
};
5+
use crate::utils::sym;
56
use if_chain::if_chain;
67
use rustc::hir::*;
78
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
@@ -37,10 +38,10 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for ByteCount {
3738
fn check_expr(&mut self, cx: &LateContext<'_, '_>, expr: &Expr) {
3839
if_chain! {
3940
if let ExprKind::MethodCall(ref count, _, ref count_args) = expr.node;
40-
if count.ident.name == "count";
41+
if count.ident.name == *sym::count;
4142
if count_args.len() == 1;
4243
if let ExprKind::MethodCall(ref filter, _, ref filter_args) = count_args[0].node;
43-
if filter.ident.name == "filter";
44+
if filter.ident.name == *sym::filter;
4445
if filter_args.len() == 2;
4546
if let ExprKind::Closure(_, _, body_id, _, _) = filter_args[1].node;
4647
then {
@@ -52,7 +53,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for ByteCount {
5253
if op.node == BinOpKind::Eq;
5354
if match_type(cx,
5455
walk_ptrs_ty(cx.tables.expr_ty(&filter_args[0])),
55-
&paths::SLICE_ITER);
56+
&*paths::SLICE_ITER);
5657
then {
5758
let needle = match get_path_name(l) {
5859
Some(name) if check_arg(name, argname, r) => r,
@@ -67,7 +68,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for ByteCount {
6768
let haystack = if let ExprKind::MethodCall(ref path, _, ref args) =
6869
filter_args[0].node {
6970
let p = path.ident.name;
70-
if (p == "iter" || p == "iter_mut") && args.len() == 1 {
71+
if (p == *sym::iter || p == *sym::iter_mut) && args.len() == 1 {
7172
&args[0]
7273
} else {
7374
&filter_args[0]

‎clippy_lints/src/cargo_common_metadata.rs

+1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
//! lint on missing cargo common metadata
22
33
use crate::utils::span_lint;
4+
use crate::utils::sym;
45
use rustc::lint::{EarlyContext, EarlyLintPass, LintArray, LintPass};
56
use rustc::{declare_lint_pass, declare_tool_lint};
67
use syntax::{ast::*, source_map::DUMMY_SP};

‎clippy_lints/src/cognitive_complexity.rs

+3-2
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ use rustc::{declare_tool_lint, impl_lint_pass};
99
use syntax::ast::Attribute;
1010
use syntax::source_map::Span;
1111

12+
use crate::utils::sym;
1213
use crate::utils::{in_macro_or_desugar, is_allowed, match_type, paths, span_help_and_lint, LimitStack};
1314

1415
declare_clippy_lint! {
@@ -71,7 +72,7 @@ impl CognitiveComplexity {
7172
..
7273
} = helper;
7374
let ret_ty = cx.tables.node_type(expr.hir_id);
74-
let ret_adjust = if match_type(cx, ret_ty, &paths::RESULT) {
75+
let ret_adjust = if match_type(cx, ret_ty, &*paths::RESULT) {
7576
returns
7677
} else {
7778
returns / 2
@@ -118,7 +119,7 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for CognitiveComplexity {
118119
hir_id: HirId,
119120
) {
120121
let def_id = cx.tcx.hir().local_def_id_from_hir_id(hir_id);
121-
if !cx.tcx.has_attr(def_id, "test") {
122+
if !cx.tcx.has_attr(def_id, *sym::test) {
122123
self.check(cx, body, span);
123124
}
124125
}

‎clippy_lints/src/collapsible_if.rs

+1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ use rustc::lint::{EarlyContext, EarlyLintPass, LintArray, LintPass};
1717
use rustc::{declare_lint_pass, declare_tool_lint};
1818
use syntax::ast;
1919

20+
use crate::utils::sym;
2021
use crate::utils::sugg::Sugg;
2122
use crate::utils::{
2223
in_macro_or_desugar, snippet_block, snippet_block_with_applicability, span_lint_and_sugg, span_lint_and_then,

‎clippy_lints/src/const_static_lifetime.rs

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
use crate::utils::{in_macro_or_desugar, snippet, span_lint_and_then};
2+
use crate::utils::sym;
23
use rustc::lint::{EarlyContext, EarlyLintPass, LintArray, LintPass};
34
use rustc::{declare_lint_pass, declare_tool_lint};
45
use rustc_errors::Applicability;
@@ -47,7 +48,7 @@ impl StaticConst {
4748
if let Some(lifetime) = *optional_lifetime {
4849
match borrow_type.ty.node {
4950
TyKind::Path(..) | TyKind::Slice(..) | TyKind::Array(..) | TyKind::Tup(..) => {
50-
if lifetime.ident.name == "'static" {
51+
if lifetime.ident.name == syntax::symbol::keywords::StaticLifetime.name() {
5152
let snip = snippet(cx, borrow_type.ty.span, "<type>");
5253
let sugg = format!("&{}", snip);
5354
span_lint_and_then(

‎clippy_lints/src/consts.rs

+1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
#![allow(clippy::float_cmp)]
22

33
use crate::utils::{clip, higher, sext, unsext};
4+
use crate::utils::sym;
45
use if_chain::if_chain;
56
use rustc::hir::def::{DefKind, Res};
67
use rustc::hir::*;

‎clippy_lints/src/copies.rs

+1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
use crate::utils::{get_parent_expr, higher, in_macro_or_desugar, snippet, span_lint_and_then, span_note_and_lint};
22
use crate::utils::{SpanlessEq, SpanlessHash};
3+
use crate::utils::sym;
34
use rustc::hir::*;
45
use rustc::lint::{LateContext, LateLintPass, LintArray, LintPass};
56
use rustc::ty::Ty;

0 commit comments

Comments
 (0)