chore: remove a couple of unwraps, replace with expects

Walther · Nov 26, 2024 · e50c4fd · e50c4fd
1 parent 4f5010e
commit e50c4fd
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/clovers/src/hitable.rs b/clovers/src/hitable.rs
@@ -143,7 +143,7 @@ impl<'scene> HitableList<'scene> {
     /// This method may panic if no finite bounding box can be created for the given `hitables`.
     #[must_use]
     pub fn new(hitables: Vec<Hitable<'scene>>) -> Self {
-        let aabb = vec_bounding_box(&hitables).unwrap();
+        let aabb = vec_bounding_box(&hitables).expect("No bounding box for hitables");
         Self { hitables, aabb }
     }
 

diff --git a/clovers/src/objects/gltf.rs b/clovers/src/objects/gltf.rs
@@ -72,7 +72,7 @@ impl<'scene> GLTF<'scene> {
     /// This method may panic if no finite bounding box can be created for the given `hitables`.
     pub fn new(gltf_init: GLTFInit) -> Self {
         let hitables: Vec<Hitable> = gltf_init.into();
-        let aabb = vec_bounding_box(&hitables).unwrap();
+        let aabb = vec_bounding_box(&hitables).expect("No bounding box for hitables");
 
         GLTF { hitables, aabb }
     }

diff --git a/clovers/src/objects/ply.rs b/clovers/src/objects/ply.rs
@@ -108,7 +108,7 @@ pub fn initialize_ply<'scene>(
         hitables.push(Hitable::Triangle(triangle));
     }
     // TODO: remove unwrap
-    let aabb = vec_bounding_box(&hitables).unwrap();
+    let aabb = vec_bounding_box(&hitables).expect("No bounding box for hitables");
 
     PLY {
         hitables,