From 9eb9fcd71962fae50c63431d63caab923c68182e Mon Sep 17 00:00:00 2001 From: Jimmie Elvenmark Date: Sun, 24 Aug 2014 09:07:19 +0200 Subject: [PATCH] lint: Improve camel case suggestion when empty. --- src/librustc/lint/builtin.rs | 10 +++++++--- src/test/compile-fail/lint-non-camel-case-types.rs | 2 +- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/src/librustc/lint/builtin.rs b/src/librustc/lint/builtin.rs index 1fa4c53832f..68061a94dc8 100644 --- a/src/librustc/lint/builtin.rs +++ b/src/librustc/lint/builtin.rs @@ -768,9 +768,13 @@ impl LintPass for NonCamelCaseTypes { let s = token::get_ident(ident); if !is_camel_case(ident) { - cx.span_lint(NON_CAMEL_CASE_TYPES, span, - format!("{} `{}` should have a camel case name such as `{}`", - sort, s, to_camel_case(s.get())).as_slice()); + let c = to_camel_case(s.get()); + let m = if c.is_empty() { + format!("{} `{}` should have a camel case name such as `CamelCase`", sort, s) + } else { + format!("{} `{}` should have a camel case name such as `{}`", sort, s, c) + }; + cx.span_lint(NON_CAMEL_CASE_TYPES, span, m.as_slice()); } } diff --git a/src/test/compile-fail/lint-non-camel-case-types.rs b/src/test/compile-fail/lint-non-camel-case-types.rs index 03a2d63e123..f33c7956a14 100644 --- a/src/test/compile-fail/lint-non-camel-case-types.rs +++ b/src/test/compile-fail/lint-non-camel-case-types.rs @@ -37,6 +37,6 @@ struct foo7 { bar: int, } -type __ = int; //~ ERROR type `__` should have a camel case name such as `` +type __ = int; //~ ERROR type `__` should have a camel case name such as `CamelCase` fn main() { }