From ba643fadfea11fe137f0fa231384fda1379c05ce Mon Sep 17 00:00:00 2001
From: Andre Bogus <bogusandre@gmail.com>
Date: Sat, 26 Aug 2017 22:18:04 +0200
Subject: [PATCH] add a lowercase suggestion to unknown_lints

---
 src/librustc/lint/levels.rs       | 20 +++++++++++++++++---
 src/test/ui/lint/not_found.rs     | 21 +++++++++++++++++++++
 src/test/ui/lint/not_found.stderr | 20 ++++++++++++++++++++
 3 files changed, 58 insertions(+), 3 deletions(-)
 create mode 100644 src/test/ui/lint/not_found.rs
 create mode 100644 src/test/ui/lint/not_found.stderr

diff --git a/src/librustc/lint/levels.rs b/src/librustc/lint/levels.rs
index ab086e5b8e90e..c5863b5618feb 100644
--- a/src/librustc/lint/levels.rs
+++ b/src/librustc/lint/levels.rs
@@ -247,13 +247,27 @@ impl<'a> LintLevelsBuilder<'a> {
                                                                     self.cur,
                                                                     Some(&specs));
                         let msg = format!("unknown lint: `{}`", name);
-                        lint::struct_lint_level(self.sess,
+                        let mut db = lint::struct_lint_level(self.sess,
                                                 lint,
                                                 level,
                                                 src,
                                                 Some(li.span.into()),
-                                                &msg)
-                            .emit();
+                                                &msg);
+                        if name.as_str().chars().any(|c| c.is_uppercase()) {
+                            let name_lower = name.as_str().to_lowercase();
+                            if let CheckLintNameResult::NoLint =
+                                    store.check_lint_name(&name_lower) {
+                                db.emit();
+                            } else {
+                                db.span_suggestion(
+                                    li.span,
+                                    "lowercase the lint name",
+                                    name_lower
+                                ).emit();
+                            }
+                        } else {
+                            db.emit();
+                        }
                     }
                 }
             }
diff --git a/src/test/ui/lint/not_found.rs b/src/test/ui/lint/not_found.rs
new file mode 100644
index 0000000000000..5cdc19823cfff
--- /dev/null
+++ b/src/test/ui/lint/not_found.rs
@@ -0,0 +1,21 @@
+// Copyright 2014–2017 The Rust Project Developers. See the COPYRIGHT
+// file at the top-level directory of this distribution and at
+// http://rust-lang.org/COPYRIGHT.
+//
+// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+// this tests the `unknown_lint` lint, especially the suggestions
+
+// the suggestion only appears if a lint with the lowercase name exists
+#[allow(FOO_BAR)]
+// the suggestion appears on all-uppercase names
+#[warn(DEAD_CODE)]
+// the suggestion appears also on mixed-case names
+#[deny(Warnings)]
+fn main() {
+    unimplemented!();
+}
diff --git a/src/test/ui/lint/not_found.stderr b/src/test/ui/lint/not_found.stderr
new file mode 100644
index 0000000000000..73265845494c2
--- /dev/null
+++ b/src/test/ui/lint/not_found.stderr
@@ -0,0 +1,20 @@
+warning: unknown lint: `FOO_BAR`
+  --> $DIR/not_found.rs:14:9
+   |
+14 | #[allow(FOO_BAR)]
+   |         ^^^^^^^
+   |
+   = note: #[warn(unknown_lints)] on by default
+
+warning: unknown lint: `DEAD_CODE`
+  --> $DIR/not_found.rs:16:8
+   |
+16 | #[warn(DEAD_CODE)]
+   |        ^^^^^^^^^ help: lowercase the lint name: `dead_code`
+
+warning: unknown lint: `Warnings`
+  --> $DIR/not_found.rs:18:8
+   |
+18 | #[deny(Warnings)]
+   |        ^^^^^^^^ help: lowercase the lint name: `warnings`
+