Skip to content

Commit

Permalink
resolve: make error message strings more consistent
Browse files Browse the repository at this point in the history
The "first definition of ..." error string reported by add_child() looks
different from similar messages reported by other functions.  Fix this.

Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
  • Loading branch information
artagnon committed Jun 25, 2013
1 parent 9f80591 commit f982f42
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/librustc/middle/resolve.rs
Expand Up @@ -1053,7 +1053,7 @@ impl Resolver {
let r = child.span_for_namespace(ns);
for r.iter().advance |sp| {
self.session.span_note(*sp,
fmt!("first definition of %s %s here:",
fmt!("first definition of %s `%s` here",
namespace_error_to_str(duplicate_type),
self.session.str_of(name)));
}
Expand Down

5 comments on commit f982f42

@bors
Copy link
Contributor

@bors bors commented on f982f42 Jun 25, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on f982f42 Jun 25, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging artagnon/rust/resolve-module = f982f42 into auto

@bors
Copy link
Contributor

@bors bors commented on f982f42 Jun 25, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

artagnon/rust/resolve-module = f982f42 merged ok, testing candidate = 4967bd0

@bors
Copy link
Contributor

@bors bors commented on f982f42 Jun 25, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 4967bd0

Please sign in to comment.