Skip to content

Commit 054687b

Browse files
---
yaml --- r: 220147 b: refs/heads/snap-stage3 c: c5f7c19 h: refs/heads/master i: 220145: 4396e41 220143: 27c9d91 v: v3
1 parent b489fae commit 054687b

File tree

4 files changed

+431
-190
lines changed

4 files changed

+431
-190
lines changed

[refs]

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
refs/heads/master: c044791d80ea0dc5c4b57b6030a67b69f8510239
3-
refs/heads/snap-stage3: f52a87c44e84a0089819e77ad85ea24b88d547e7
3+
refs/heads/snap-stage3: c5f7c19cf274f510b48d6835f5677ae22e978d6d
44
refs/heads/try: b53c0f93eedcdedd4fd89bccc5a3a09d1c5cd23e
55
refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
66
refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596

branches/snap-stage3/src/librustc_resolve/build_reduced_graph.rs

Lines changed: 22 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -208,10 +208,13 @@ impl<'a, 'b:'a, 'tcx:'b> GraphBuilder<'a, 'b, 'tcx> {
208208
// Return an error here by looking up the namespace that
209209
// had the duplicate.
210210
let ns = ns.unwrap();
211-
resolve_err!(self, sp, E0428,
212-
"duplicate definition of {} `{}`",
213-
namespace_error_to_string(duplicate_type),
214-
token::get_name(name));
211+
::resolve_error(
212+
&::ResolutionError::DuplicateDefinition(
213+
self,
214+
sp,
215+
namespace_error_to_string(duplicate_type),
216+
&*token::get_name(name))
217+
);
215218
{
216219
let r = child.span_for_namespace(ns);
217220
if let Some(sp) = r {
@@ -304,9 +307,10 @@ impl<'a, 'b:'a, 'tcx:'b> GraphBuilder<'a, 'b, 'tcx> {
304307
full_path.segments.last().unwrap().identifier.name;
305308
if &token::get_name(source_name)[..] == "mod" ||
306309
&token::get_name(source_name)[..] == "self" {
307-
resolve_err!(self, view_path.span, E0429,
308-
"{}",
309-
"`self` imports are only allowed within a { } list");
310+
::resolve_error(&::ResolutionError::SelfImportsOnlyAllowedWithin(
311+
self,
312+
view_path.span)
313+
);
310314
}
311315

312316
let subclass = SingleImport(binding.name,
@@ -326,9 +330,11 @@ impl<'a, 'b:'a, 'tcx:'b> GraphBuilder<'a, 'b, 'tcx> {
326330
_ => None
327331
}).collect::<Vec<Span>>();
328332
if mod_spans.len() > 1 {
329-
resolve_err!(self, mod_spans[0], E0430,
330-
"{}",
331-
"`self` import can only appear once in the list");
333+
::resolve_error(
334+
&::ResolutionError::SelfImportCanOnlyAppearOnceInTheList(
335+
self,
336+
mod_spans[0])
337+
);
332338
for other_span in mod_spans.iter().skip(1) {
333339
self.session.span_note(*other_span,
334340
"another `self` import appears here");
@@ -343,10 +349,12 @@ impl<'a, 'b:'a, 'tcx:'b> GraphBuilder<'a, 'b, 'tcx> {
343349
let name = match module_path.last() {
344350
Some(name) => *name,
345351
None => {
346-
resolve_err!(self, source_item.span, E0431,
347-
"{}",
348-
"`self` import can only appear in an import list \
349-
with a non-empty prefix");
352+
::resolve_error(
353+
&::ResolutionError::
354+
SelfImportOnlyInImportListWithNonEmptyPrefix(
355+
self,
356+
source_item.span)
357+
);
350358
continue;
351359
}
352360
};

0 commit comments

Comments
 (0)