migrate all hashing API users to gix_hash::Hasher::finalize()

Author: emilazy

gix-commitgraph/src/file/verify.rs

@@ -147,7 +147,7 @@ impl File {
         let data_len_without_trailer = self.data.len() - self.hash_len;
         let mut hasher = gix_hash::hasher(self.object_hash());
         hasher.update(&self.data[..data_len_without_trailer]);
-        let actual = gix_hash::ObjectId::from_bytes_or_panic(hasher.digest().as_ref());
+        let actual = hasher.finalize();
 
         let expected = self.checksum();
         if actual == expected {

gix-hash/src/hasher/io.rs

@@ -66,7 +66,7 @@ pub fn bytes_with_hasher(
         }
     }
 
-    let id = crate::ObjectId::from(hasher.digest());
+    let id = hasher.finalize();
     progress.show_throughput(start);
     Ok(id)
 }

gix-index/src/extension/end_of_index_entry/decode.rs

@@ -41,7 +41,7 @@ pub fn decode(data: &[u8], object_hash: gix_hash::Kind) -> Option<usize> {
         last_chunk = Some(chunk);
     }
 
-    if hasher.digest() != checksum {
+    if hasher.finalize().as_slice() != checksum {
         return None;
     }
     // The last-to-this chunk ends where ours starts

gix-index/src/extension/end_of_index_entry/write.rs

@@ -23,7 +23,7 @@ pub fn write_to(
         hasher.update(&signature);
         hasher.update(&size.to_be_bytes());
     }
-    out.write_all(&hasher.digest())?;
+    out.write_all(hasher.finalize().as_slice())?;
 
     Ok(())
 }

gix-index/src/file/write.rs

@@ -31,7 +31,7 @@ impl File {
             let mut hasher = hasher::io::Write::new(&mut out, self.state.object_hash);
             let out: &mut dyn std::io::Write = &mut hasher;
             let version = self.state.write_to(out, options)?;
-            (version, gix_hash::ObjectId::from(hasher.hash.digest()))
+            (version, hasher.hash.finalize())
         };
         out.write_all(hash.as_slice())?;
         Ok((version, hash))

gix-object/src/lib.rs

@@ -409,7 +409,7 @@ fn object_hasher(hash_kind: gix_hash::Kind, object_kind: Kind, object_size: u64)
 pub fn compute_hash(hash_kind: gix_hash::Kind, object_kind: Kind, data: &[u8]) -> gix_hash::ObjectId {
     let mut hasher = object_hasher(hash_kind, object_kind, data.len() as u64);
     hasher.update(data);
-    hasher.digest().into()
+    hasher.finalize()
 }
 
 /// A function to compute a hash of kind `hash_kind` for an object of `object_kind` and its data read from `stream`

gix-odb/src/sink.rs

@@ -53,6 +53,6 @@ impl gix_object::Write for Sink {
             c.reset();
         }
 
-        Ok(hasher.digest().into())
+        Ok(hasher.finalize())
     }
 }

gix-odb/src/store_impls/loose/write.rs

@@ -130,7 +130,7 @@ impl Store {
         &self,
         hasher::io::Write { hash, inner: file }: hasher::io::Write<CompressedTempfile>,
     ) -> Result<gix_hash::ObjectId, Error> {
-        let id = gix_hash::ObjectId::from(hash.digest());
+        let id = hash.finalize();
         let object_path = loose::hash_path(&id, self.path.clone());
         let object_dir = object_path
             .parent()

gix-pack/src/data/input/bytes_to_entries.rs

@@ -177,7 +177,7 @@ where
             }
 
             if let Some(hash) = self.hash.take() {
-                let actual_id = gix_hash::ObjectId::from(hash.digest());
+                let actual_id = hash.finalize();
                 if self.mode == input::Mode::Restore {
                     id = actual_id;
                 }
@@ -191,7 +191,7 @@ where
             Some(id)
         } else if self.mode == input::Mode::Restore {
             let hash = self.hash.clone().expect("in restore mode a hash is set");
-            Some(gix_hash::ObjectId::from(hash.digest()))
+            Some(hash.finalize())
         } else {
             None
         })

gix-pack/src/data/output/bytes.rs

@@ -121,12 +121,12 @@ where
                 }
             }
             None => {
-                let digest = self.output.hash.clone().digest();
-                self.output.inner.write_all(&digest[..])?;
-                self.written += digest.len() as u64;
+                let digest = self.output.hash.clone().finalize();
+                self.output.inner.write_all(digest.as_slice())?;
+                self.written += digest.as_slice().len() as u64;
                 self.output.inner.flush()?;
                 self.is_done = true;
-                self.trailer = Some(gix_hash::ObjectId::from(digest));
+                self.trailer = Some(digest);
             }
         }
         Ok(self.written - previous_written)