about summary refs log tree commit diff
diff options
context:
space:
mode:
authorAlex Crichton <alex@alexcrichton.com>2014-07-10 10:19:10 -0700
committerAlex Crichton <alex@alexcrichton.com>2014-07-10 10:19:10 -0700
commit6bd79d32e98dfe433c525a59ec69a6e5d77f293d (patch)
tree18808e9d496dfa7c8df037aebd1180075964da5e
parentf9fe251777e9f1cc557a6d5b82b45403935d0a10 (diff)
downloadrust-6bd79d32e98dfe433c525a59ec69a6e5d77f293d.tar.gz
rust-6bd79d32e98dfe433c525a59ec69a6e5d77f293d.zip
rustc: Always mark #[crate_name] as used
It's just an annoying error if you use --crate-name on the command line and you
also have a #[crate_name] specified
-rw-r--r--src/librustc/back/link.rs10
-rw-r--r--src/test/run-pass/crate-name-attr-used.rs15
2 files changed, 22 insertions, 3 deletions
diff --git a/src/librustc/back/link.rs b/src/librustc/back/link.rs
index e2264088b4a..a1df360a11b 100644
--- a/src/librustc/back/link.rs
+++ b/src/librustc/back/link.rs
@@ -555,6 +555,12 @@ pub fn find_crate_name(sess: Option<&Session>,
         s
     };
 
+    // Look in attributes 100% of the time to make sure the attribute is marked
+    // as used. After doing this, however, favor crate names from the command
+    // line.
+    let attr_crate_name = attrs.iter().find(|at| at.check_name("crate_name"))
+                               .and_then(|at| at.value_str().map(|s| (at, s)));
+
     match sess {
         Some(sess) => {
             match sess.opts.crate_name {
@@ -565,9 +571,7 @@ pub fn find_crate_name(sess: Option<&Session>,
         None => {}
     }
 
-    let crate_name = attrs.iter().find(|at| at.check_name("crate_name"))
-                          .and_then(|at| at.value_str().map(|s| (at, s)));
-    match crate_name {
+    match attr_crate_name {
         Some((attr, s)) => return validate(s.get().to_string(), Some(attr.span)),
         None => {}
     }
diff --git a/src/test/run-pass/crate-name-attr-used.rs b/src/test/run-pass/crate-name-attr-used.rs
new file mode 100644
index 00000000000..abc565d3175
--- /dev/null
+++ b/src/test/run-pass/crate-name-attr-used.rs
@@ -0,0 +1,15 @@
+// Copyright 2014 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.
+
+// compile-flags:--crate-name crate-name-attr-used -F unused-attribute
+
+#![crate_name = "test"]
+
+fn main() {}