recursor: rename Recursor::new() to builder()

hickory-dns · May 12, 2024 · 695be39 · 695be39
1 parent ba29c88
commit 695be39
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/crates/recursor/src/recursor.rs b/crates/recursor/src/recursor.rs
@@ -108,9 +108,8 @@ pub struct Recursor {
 }
 
 impl Recursor {
-    /// Short-hand for `RecursorBuilder::default`
-    #[allow(clippy::new_ret_no_self)]
-    pub fn new() -> RecursorBuilder {
+    /// Construct the new [`Recursor`] via the [`RecursorBuilder`]
+    pub fn builder() -> RecursorBuilder {
         RecursorBuilder::default()
     }
 

diff --git a/crates/server/src/store/recursor/authority.rs b/crates/server/src/store/recursor/authority.rs
@@ -73,7 +73,7 @@ impl RecursiveAuthority {
             });
         }
 
-        let mut recursor = Recursor::new();
+        let mut recursor = Recursor::builder();
         recursor
             .ns_cache_size(config.ns_cache_size)
             .record_cache_size(config.record_cache_size);

diff --git a/util/src/bin/recurse.rs b/util/src/bin/recurse.rs
@@ -157,7 +157,7 @@ pub async fn main() -> Result<(), Box<dyn std::error::Error>> {
     let name = opts.domainname;
     let ty = opts.ty;
 
-    let recursor = Recursor::new().build(roots)?;
+    let recursor = Recursor::builder().build(roots)?;
 
     // execute query
     println!(