🐙 Fixed negative orgin subimage panic.

Author: rizerco

src/image.rs

@@ -593,6 +593,14 @@ impl Image {
     /// Returns a new image that is a subimage of this image within
     /// the supplied bounds.
     pub fn subimage(&self, region: Rect<i32>) -> anyhow::Result<Image> {
+        let bounds = Rect {
+            origin: Point::zero(),
+            size: self.size,
+        }
+        .into();
+        let Some(region) = region.intersection(&bounds) else {
+            anyhow::bail!(ImageError::OutOfBounds)
+        };
         let x = region.min_x() as u32;
         let y = region.min_y() as u32;
         let width = region.width() as u32;

tests/image_tests.rs

@@ -135,12 +135,25 @@ mod tests {
     #[test]
     fn subimage() {
         let image = Image::open("tests/images/avatar.png").unwrap();
-        let expected_image = Image::open("tests/images/avatar_subimage.png").unwrap();
+        let expected_image = Image::open("tests/images/avatar-subimage.png").unwrap();
 
         let region = Rect::new(2, 7, 13, 10);
         let result = image.subimage(region).unwrap();
 
-        // result.save("/tmp/avatar_subimage.png").unwrap();
+        // result.save("/tmp/avatar-subimage.png").unwrap();
+
+        assert!(result.appears_equal_to(&expected_image));
+    }
+
+    #[test]
+    fn negative_subimage() {
+        let image = Image::open("tests/images/avatar.png").unwrap();
+        let expected_image = Image::open("tests/images/avatar-negative-subimage.png").unwrap();
+
+        let region = Rect::new(-2, 7, 13, 10);
+        let result = image.subimage(region).unwrap();
+
+        // result.save("/tmp/avatar-negative-subimage.png").unwrap();
 
         assert!(result.appears_equal_to(&expected_image));
     }