CheckContext -> &CheckContext

CHANGELOG.md

@@ -6,7 +6,12 @@
 
 * Warnings from cargo are no longer silenced when documenting. [PR#114]
 
+#### Changed
+
+* `walk_dir` now takes `&CheckContext`, not `CheckContext`. [PR#118]
+
 [PR#114]: https://github.com/deadlinks/cargo-deadlinks/pull/114
+[PR#118]: https://github.com/deadlinks/cargo-deadlinks/pull/118
 
 <a name="0.6.1"></a>
 ## 0.6.1 (2020-11-23)

src/bin/cargo-deadlinks.rs

@@ -83,7 +83,7 @@ fn main() {
             }
         };
         log::info!("checking directory {:?}", dir);
-        if walk_dir(&dir, ctx.clone()) {
+        if walk_dir(&dir, &ctx) {
             errors = true;
         }
     }

src/bin/deadlinks.rs

@@ -58,7 +58,7 @@ fn main() {
         }
     };
     log::info!("checking directory {:?}", dir);
-    if walk_dir(&dir, args.into()) {
+    if walk_dir(&dir, &args.into()) {
         process::exit(1);
     }
 }

src/lib.rs

@@ -50,14 +50,14 @@ impl fmt::Display for FileError {
 ///
 /// For each error that occurred, print an error message.
 /// Returns whether an error occurred.
-pub fn walk_dir(dir_path: &Path, ctx: CheckContext) -> bool {
+pub fn walk_dir(dir_path: &Path, ctx: &CheckContext) -> bool {
     let pool = ThreadPoolBuilder::new()
         .num_threads(num_cpus::get())
         .build()
         .unwrap();
 
     pool.install(|| {
-        unavailable_urls(dir_path, &ctx)
+        unavailable_urls(dir_path, ctx)
             .map(|err| {
                 if ctx.verbose {
                     println!("{}", err);