FilePatcher: return None when num_replacements is 0

Fix #102
your-tools · Dec 3, 2022 · ebbcfbe · ebbcfbe
1 parent c749858
commit ebbcfbe
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/src/file_patcher.rs b/src/file_patcher.rs
@@ -62,12 +62,16 @@ impl FilePatcher {
                 }
             }
         }
-        Ok(Some(FilePatcher {
-            path: path.to_path_buf(),
-            new_contents,
-            num_lines,
-            num_replacements,
-        }))
+        if num_replacements == 0 {
+            Ok(None)
+        } else {
+            Ok(Some(FilePatcher {
+                path: path.to_path_buf(),
+                new_contents,
+                num_lines,
+                num_replacements,
+            }))
+        }
     }
 
     pub(crate) fn num_replacements(&self) -> usize {