Arguments are now passed directly to the function instead of the enum variants

src/librustc_resolve/build_reduced_graph.rs

@@ -209,9 +209,9 @@ fn add_child(&self,
                     // had the duplicate.
                     let ns = ns.unwrap();
                     ::resolve_error(
-                        &::ResolutionError::DuplicateDefinition(
                         self,
                         sp,
+                        &::ResolutionError::DuplicateDefinition(
                             namespace_error_to_string(duplicate_type),
                             &*token::get_name(name))
                     );
@@ -307,9 +307,9 @@ fn build_reduced_graph_for_item(&mut self, item: &Item, parent: &Rc<Module>) ->
                             full_path.segments.last().unwrap().identifier.name;
                         if &token::get_name(source_name)[..] == "mod" ||
                            &token::get_name(source_name)[..] == "self" {
-                            ::resolve_error(&::ResolutionError::SelfImportsOnlyAllowedWithin(
-                                                                                self,
-                                                                                view_path.span)
+                            ::resolve_error(self,
+                                            view_path.span,
+                                            &::ResolutionError::SelfImportsOnlyAllowedWithin
                             );
                         }
 
@@ -331,9 +331,9 @@ fn build_reduced_graph_for_item(&mut self, item: &Item, parent: &Rc<Module>) ->
                         }).collect::<Vec<Span>>();
                         if mod_spans.len() > 1 {
                             ::resolve_error(
-                                &::ResolutionError::SelfImportCanOnlyAppearOnceInTheList(
                                 self,
-                                    mod_spans[0])
+                                mod_spans[0],
+                                &::ResolutionError::SelfImportCanOnlyAppearOnceInTheList
                             );
                             for other_span in mod_spans.iter().skip(1) {
                                 self.session.span_note(*other_span,
@@ -350,10 +350,10 @@ fn build_reduced_graph_for_item(&mut self, item: &Item, parent: &Rc<Module>) ->
                                         Some(name) => *name,
                                         None => {
                                             ::resolve_error(
-                                                &::ResolutionError::
-                                                SelfImportOnlyInImportListWithNonEmptyPrefix(
                                                 self,
-                                                    source_item.span)
+                                                source_item.span,
+                                                &::ResolutionError::
+                                                SelfImportOnlyInImportListWithNonEmptyPrefix
                                             );
                                             continue;
                                         }

src/librustc_resolve/resolve_imports.rs

@@ -272,7 +272,9 @@ fn resolve_imports_for_module(&mut self, module: Rc<Module>) {
                         Some((span, msg)) => (span, format!(". {}", msg)),
                         None => (import_directive.span, String::new())
                     };
-                    ::resolve_error(&::ResolutionError::UnresolvedImport(self.resolver, span,
+                    ::resolve_error(self.resolver,
+                                    span,
+                                    &::ResolutionError::UnresolvedImport(
                                                 Some((&*import_path_to_string(
                                                         &import_directive.module_path,
                                                         import_directive.subclass),