1 // Copyright 2015 The Rust Project Developers. See the COPYRIGHT
2 // file at the top-level directory of this distribution and at
3 // http://rust-lang.org/COPYRIGHT.
5 // Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
6 // http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
7 // <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
8 // option. This file may not be copied, modified, or distributed
9 // except according to those terms.
11 use rustc
::hir
::def
::Def
;
13 use lint
::{LateContext, LintContext, LintArray}
;
14 use lint
::{LintPass, LateLintPass}
;
16 use rustc_target
::spec
::abi
::Abi
;
21 use rustc
::hir
::{self, GenericParamKind, PatKind}
;
22 use rustc
::hir
::intravisit
::FnKind
;
25 pub enum MethodLateContext
{
31 pub fn method_context(cx
: &LateContext
, id
: ast
::NodeId
) -> MethodLateContext
{
32 let def_id
= cx
.tcx
.hir
.local_def_id(id
);
33 let item
= cx
.tcx
.associated_item(def_id
);
34 match item
.container
{
35 ty
::TraitContainer(..) => MethodLateContext
::TraitAutoImpl
,
36 ty
::ImplContainer(cid
) => {
37 match cx
.tcx
.impl_trait_ref(cid
) {
38 Some(_
) => MethodLateContext
::TraitImpl
,
39 None
=> MethodLateContext
::PlainImpl
,
46 pub NON_CAMEL_CASE_TYPES
,
48 "types, variants, traits and type parameters should have camel case names"
51 #[derive(Copy, Clone)]
52 pub struct NonCamelCaseTypes
;
54 impl NonCamelCaseTypes
{
55 fn check_case(&self, cx
: &LateContext
, sort
: &str, name
: ast
::Name
, span
: Span
) {
56 fn char_has_case(c
: char) -> bool
{
57 c
.is_lowercase() || c
.is_uppercase()
60 fn is_camel_case(name
: ast
::Name
) -> bool
{
61 let name
= name
.as_str();
62 let name
= name
.trim_matches('_'
);
67 // start with a non-lowercase letter rather than non-uppercase
68 // ones (some scripts don't have a concept of upper/lowercase)
69 !name
.is_empty() && !name
.chars().next().unwrap().is_lowercase() &&
70 !name
.contains("__") && !name
.chars().collect
::<Vec
<_
>>().windows(2).any(|pair
| {
71 // contains a capitalisable character followed by, or preceded by, an underscore
72 char_has_case(pair
[0]) && pair
[1] == '_'
||
73 char_has_case(pair
[1]) && pair
[0] == '_'
77 fn to_camel_case(s
: &str) -> String
{
81 word
.chars().enumerate().map(|(i
, c
)| if i
== 0 {
82 c
.to_uppercase().collect
::<String
>()
84 c
.to_lowercase().collect()
88 .filter(|x
| !x
.is_empty())
89 .fold((String
::new(), None
), |(acc
, prev
): (String
, Option
<String
>), next
| {
90 // separate two components with an underscore if their boundary cannot
91 // be distinguished using a uppercase/lowercase case distinction
92 let join
= if let Some(prev
) = prev
{
93 let l
= prev
.chars().last().unwrap();
94 let f
= next
.chars().next().unwrap();
95 !char_has_case(l
) && !char_has_case(f
)
97 (acc
+ if join { "_" }
else { "" }
+ &next
, Some(next
))
101 if !is_camel_case(name
) {
102 let c
= to_camel_case(&name
.as_str());
103 let m
= if c
.is_empty() {
104 format
!("{} `{}` should have a camel case name such as `CamelCase`", sort
, name
)
106 format
!("{} `{}` should have a camel case name such as `{}`", sort
, name
, c
)
108 cx
.span_lint(NON_CAMEL_CASE_TYPES
, span
, &m
);
113 impl LintPass
for NonCamelCaseTypes
{
114 fn get_lints(&self) -> LintArray
{
115 lint_array
!(NON_CAMEL_CASE_TYPES
)
119 impl<'a
, 'tcx
> LateLintPass
<'a
, 'tcx
> for NonCamelCaseTypes
{
120 fn check_item(&mut self, cx
: &LateContext
, it
: &hir
::Item
) {
121 let has_repr_c
= it
.attrs
124 attr
::find_repr_attrs(cx
.tcx
.sess
.diagnostic(), attr
)
126 .any(|r
| r
== &attr
::ReprC
)
134 hir
::ItemKind
::Ty(..) |
135 hir
::ItemKind
::Enum(..) |
136 hir
::ItemKind
::Struct(..) |
137 hir
::ItemKind
::Union(..) => self.check_case(cx
, "type", it
.name
, it
.span
),
138 hir
::ItemKind
::Trait(..) => self.check_case(cx
, "trait", it
.name
, it
.span
),
143 fn check_variant(&mut self, cx
: &LateContext
, v
: &hir
::Variant
, _
: &hir
::Generics
) {
144 self.check_case(cx
, "variant", v
.node
.name
, v
.span
);
147 fn check_generic_param(&mut self, cx
: &LateContext
, param
: &hir
::GenericParam
) {
149 GenericParamKind
::Lifetime { .. }
=> {}
150 GenericParamKind
::Type { synthetic, .. }
=> {
151 if synthetic
.is_none() {
152 self.check_case(cx
, "type parameter", param
.name
.ident().name
, param
.span
);
162 "variables, methods, functions, lifetime parameters and modules should have snake case names"
165 #[derive(Copy, Clone)]
166 pub struct NonSnakeCase
;
169 fn to_snake_case(mut str: &str) -> String
{
170 let mut words
= vec
![];
171 // Preserve leading underscores
172 str = str.trim_left_matches(|c
: char| {
174 words
.push(String
::new());
180 for s
in str.split('_'
) {
181 let mut last_upper
= false;
182 let mut buf
= String
::new();
186 for ch
in s
.chars() {
187 if !buf
.is_empty() && buf
!= "'" && ch
.is_uppercase() && !last_upper
{
191 last_upper
= ch
.is_uppercase();
192 buf
.extend(ch
.to_lowercase());
199 fn check_snake_case(&self, cx
: &LateContext
, sort
: &str, name
: &str, span
: Option
<Span
>) {
200 fn is_snake_case(ident
: &str) -> bool
{
201 if ident
.is_empty() {
204 let ident
= ident
.trim_left_matches('
\''
);
205 let ident
= ident
.trim_matches('_'
);
207 let mut allow_underscore
= true;
208 ident
.chars().all(|c
| {
209 allow_underscore
= match c
{
210 '_'
if !allow_underscore
=> return false,
212 // It would be more obvious to use `c.is_lowercase()`,
213 // but some characters do not have a lowercase form
214 c
if !c
.is_uppercase() => true,
221 if !is_snake_case(name
) {
222 let sc
= NonSnakeCase
::to_snake_case(name
);
223 let msg
= if sc
!= name
{
224 format
!("{} `{}` should have a snake case name such as `{}`",
229 format
!("{} `{}` should have a snake case name", sort
, name
)
232 Some(span
) => cx
.span_lint(NON_SNAKE_CASE
, span
, &msg
),
233 None
=> cx
.lint(NON_SNAKE_CASE
, &msg
),
239 impl LintPass
for NonSnakeCase
{
240 fn get_lints(&self) -> LintArray
{
241 lint_array
!(NON_SNAKE_CASE
)
245 impl<'a
, 'tcx
> LateLintPass
<'a
, 'tcx
> for NonSnakeCase
{
246 fn check_crate(&mut self, cx
: &LateContext
, cr
: &hir
::Crate
) {
247 let attr_crate_name
= attr
::find_by_name(&cr
.attrs
, "crate_name")
248 .and_then(|at
| at
.value_str().map(|s
| (at
, s
)));
249 if let Some(ref name
) = cx
.tcx
.sess
.opts
.crate_name
{
250 self.check_snake_case(cx
, "crate", name
, None
);
251 } else if let Some((attr
, name
)) = attr_crate_name
{
252 self.check_snake_case(cx
, "crate", &name
.as_str(), Some(attr
.span
));
256 fn check_generic_param(&mut self, cx
: &LateContext
, param
: &hir
::GenericParam
) {
258 GenericParamKind
::Lifetime { .. }
=> {
259 let name
= param
.name
.ident().as_str();
260 self.check_snake_case(cx
, "lifetime", &name
, Some(param
.span
));
262 GenericParamKind
::Type { .. }
=> {}
266 fn check_fn(&mut self,
274 FnKind
::Method(name
, ..) => {
275 match method_context(cx
, id
) {
276 MethodLateContext
::PlainImpl
=> {
277 self.check_snake_case(cx
, "method", &name
.as_str(), Some(span
))
279 MethodLateContext
::TraitAutoImpl
=> {
280 self.check_snake_case(cx
, "trait method", &name
.as_str(), Some(span
))
285 FnKind
::ItemFn(name
, _
, header
, _
, attrs
) => {
286 // Skip foreign-ABI #[no_mangle] functions (Issue #31924)
287 if header
.abi
!= Abi
::Rust
&& attr
::find_by_name(attrs
, "no_mangle").is_some() {
290 self.check_snake_case(cx
, "function", &name
.as_str(), Some(span
))
292 FnKind
::Closure(_
) => (),
296 fn check_item(&mut self, cx
: &LateContext
, it
: &hir
::Item
) {
297 if let hir
::ItemKind
::Mod(_
) = it
.node
{
298 self.check_snake_case(cx
, "module", &it
.name
.as_str(), Some(it
.span
));
302 fn check_trait_item(&mut self, cx
: &LateContext
, item
: &hir
::TraitItem
) {
303 if let hir
::TraitItemKind
::Method(_
, hir
::TraitMethod
::Required(ref pnames
)) = item
.node
{
304 self.check_snake_case(cx
,
306 &item
.ident
.as_str(),
308 for param_name
in pnames
{
309 self.check_snake_case(cx
, "variable", ¶m_name
.as_str(), Some(param_name
.span
));
314 fn check_pat(&mut self, cx
: &LateContext
, p
: &hir
::Pat
) {
315 if let &PatKind
::Binding(_
, _
, ref ident
, _
) = &p
.node
{
316 self.check_snake_case(cx
, "variable", &ident
.as_str(), Some(p
.span
));
320 fn check_struct_def(&mut self,
322 s
: &hir
::VariantData
,
326 for sf
in s
.fields() {
327 self.check_snake_case(cx
, "structure field", &sf
.ident
.as_str(), Some(sf
.span
));
333 pub NON_UPPER_CASE_GLOBALS
,
335 "static constants should have uppercase identifiers"
338 #[derive(Copy, Clone)]
339 pub struct NonUpperCaseGlobals
;
341 impl NonUpperCaseGlobals
{
342 fn check_upper_case(cx
: &LateContext
, sort
: &str, name
: ast
::Name
, span
: Span
) {
343 if name
.as_str().chars().any(|c
| c
.is_lowercase()) {
344 let uc
= NonSnakeCase
::to_snake_case(&name
.as_str()).to_uppercase();
346 cx
.span_lint(NON_UPPER_CASE_GLOBALS
,
348 &format
!("{} `{}` should have an upper case name such as `{}`",
353 cx
.span_lint(NON_UPPER_CASE_GLOBALS
,
355 &format
!("{} `{}` should have an upper case name", sort
, name
));
361 impl LintPass
for NonUpperCaseGlobals
{
362 fn get_lints(&self) -> LintArray
{
363 lint_array
!(NON_UPPER_CASE_GLOBALS
)
367 impl<'a
, 'tcx
> LateLintPass
<'a
, 'tcx
> for NonUpperCaseGlobals
{
368 fn check_item(&mut self, cx
: &LateContext
, it
: &hir
::Item
) {
370 hir
::ItemKind
::Static(..) => {
371 if attr
::find_by_name(&it
.attrs
, "no_mangle").is_some() {
374 NonUpperCaseGlobals
::check_upper_case(cx
, "static variable", it
.name
, it
.span
);
376 hir
::ItemKind
::Const(..) => {
377 NonUpperCaseGlobals
::check_upper_case(cx
, "constant", it
.name
, it
.span
);
383 fn check_trait_item(&mut self, cx
: &LateContext
, ti
: &hir
::TraitItem
) {
385 hir
::TraitItemKind
::Const(..) => {
386 NonUpperCaseGlobals
::check_upper_case(cx
, "associated constant",
387 ti
.ident
.name
, ti
.span
);
393 fn check_impl_item(&mut self, cx
: &LateContext
, ii
: &hir
::ImplItem
) {
395 hir
::ImplItemKind
::Const(..) => {
396 NonUpperCaseGlobals
::check_upper_case(cx
, "associated constant",
397 ii
.ident
.name
, ii
.span
);
403 fn check_pat(&mut self, cx
: &LateContext
, p
: &hir
::Pat
) {
404 // Lint for constants that look like binding identifiers (#7526)
405 if let PatKind
::Path(hir
::QPath
::Resolved(None
, ref path
)) = p
.node
{
406 if let Def
::Const(..) = path
.def
{
407 if path
.segments
.len() == 1 {
408 NonUpperCaseGlobals
::check_upper_case(cx
,
409 "constant in pattern",
410 path
.segments
[0].ident
.name
,