]> git.lizzy.rs Git - rust.git/commitdiff
Include import name in import shadowing error messages.
authorBrian Koropoff <bkoropoff@gmail.com>
Tue, 19 Aug 2014 07:07:30 +0000 (00:07 -0700)
committerBrian Koropoff <bkoropoff@gmail.com>
Tue, 19 Aug 2014 07:08:44 +0000 (00:08 -0700)
This partially alleviates the confusing behavior in issue #16597

src/librustc/middle/resolve.rs

index bd779b865d69516cbfc8297d0517fc48a85a4b49..b3389cbef641f9fdac7cc8623a6ea7d52c618fcb 100644 (file)
@@ -2820,9 +2820,10 @@ fn check_for_conflicts_between_imports_and_items(&mut self,
                  .contains_key(&name) {
             match import_resolution.type_target {
                 Some(ref target) if !target.shadowable => {
-                    self.session.span_err(import_span,
-                                          "import conflicts with imported \
-                                           crate in this module");
+                    let msg = format!("import `{}` conflicts with imported \
+                                       crate in this module",
+                                      token::get_name(name).get());
+                    self.session.span_err(import_span, msg.as_slice());
                 }
                 Some(_) | None => {}
             }
@@ -2843,9 +2844,10 @@ fn check_for_conflicts_between_imports_and_items(&mut self,
                 match *name_bindings.value_def.borrow() {
                     None => {}
                     Some(ref value) => {
-                        self.session.span_err(import_span,
-                                              "import conflicts with value \
-                                               in this module");
+                        let msg = format!("import `{}` conflicts with value \
+                                           in this module",
+                                          token::get_name(name).get());
+                        self.session.span_err(import_span, msg.as_slice());
                         match value.value_span {
                             None => {}
                             Some(span) => {
@@ -2865,9 +2867,10 @@ fn check_for_conflicts_between_imports_and_items(&mut self,
                 match *name_bindings.type_def.borrow() {
                     None => {}
                     Some(ref ty) => {
-                        self.session.span_err(import_span,
-                                              "import conflicts with type in \
-                                               this module");
+                        let msg = format!("import `{}` conflicts with type in \
+                                           this module",
+                                          token::get_name(name).get());
+                        self.session.span_err(import_span, msg.as_slice());
                         match ty.type_span {
                             None => {}
                             Some(span) => {