refactor(registry): Move query out-param to be last

Author: epage

crates/resolver-tests/src/lib.rs

@@ -128,8 +128,8 @@ pub fn resolve_with_config_raw(
         fn query(
             &mut self,
             dep: &Dependency,
-            f: &mut dyn FnMut(Summary),
             fuzzy: bool,
+            f: &mut dyn FnMut(Summary),
         ) -> Poll<CargoResult<()>> {
             for summary in self.list.iter() {
                 if fuzzy || dep.matches(summary) {

src/cargo/core/registry.rs

@@ -19,13 +19,13 @@ pub trait Registry {
     fn query(
         &mut self,
         dep: &Dependency,
-        f: &mut dyn FnMut(Summary),
         fuzzy: bool,
+        f: &mut dyn FnMut(Summary),
     ) -> Poll<CargoResult<()>>;
 
     fn query_vec(&mut self, dep: &Dependency, fuzzy: bool) -> Poll<CargoResult<Vec<Summary>>> {
         let mut ret = Vec::new();
-        self.query(dep, &mut |s| ret.push(s), fuzzy)
+        self.query(dep, fuzzy, &mut |s| ret.push(s))
             .map_ok(|()| ret)
     }
 
@@ -575,8 +575,8 @@ impl<'cfg> Registry for PackageRegistry<'cfg> {
     fn query(
         &mut self,
         dep: &Dependency,
-        f: &mut dyn FnMut(Summary),
         fuzzy: bool,
+        f: &mut dyn FnMut(Summary),
     ) -> Poll<CargoResult<()>> {
         assert!(self.patches_locked);
         let (override_summary, n, to_warn) = {

src/cargo/core/resolver/dep_cache.rs

@@ -100,13 +100,9 @@ impl<'a> RegistryQueryer<'a> {
         }
 
         let mut ret = Vec::new();
-        let ready = self.registry.query(
-            dep,
-            &mut |s| {
-                ret.push(s);
-            },
-            false,
-        )?;
+        let ready = self.registry.query(dep, false, &mut |s| {
+            ret.push(s);
+        })?;
         if ready.is_pending() {
             self.registry_cache.insert(dep.clone(), Poll::Pending);
             return Poll::Pending;