Replace line_drawing with clipline

examples/conway/Cargo.toml

@@ -14,7 +14,7 @@ byteorder = "1"
 env_logger = "0.10"
 error-iter = "0.4"
 getrandom = "0.2"
-line_drawing = "1"
+clipline = "0.1.1"
 log = "0.4"
 pixels = { path = "../.." }
 randomize = "3"

examples/conway/src/main.rs

@@ -318,19 +318,16 @@ impl ConwayGrid {
         }
     }
 
-    fn set_line(&mut self, x0: isize, y0: isize, x1: isize, y1: isize, alive: bool) {
-        // probably should do sutherland-hodgeman if this were more serious.
-        // instead just clamp the start pos, and draw until moving towards the
-        // end pos takes us out of bounds.
-        let x0 = x0.clamp(0, self.width as isize);
-        let y0 = y0.clamp(0, self.height as isize);
-        for (x, y) in line_drawing::Bresenham::new((x0, y0), (x1, y1)) {
-            if let Some(i) = self.grid_idx(x, y) {
-                self.cells[i].set_alive(alive);
-            } else {
-                break;
-            }
+    fn set_line(&mut self, x0: isize, y0: isize, x1: isize, y1: isize, alive: bool) -> Option<()> {
+        // possible to optimize by matching on Clipline and iterating over its arms
+        for (x, y) in clipline::Clipline::new(
+            ((x0, y0), (x1, y1)),
+            ((0, 0), (self.width as isize - 1, self.height as isize - 1)),
+        )? {
+            let (x, y) = (x as usize, y as usize);
+            self.cells[x + y * self.width].set_alive(alive);
         }
+        Some(())
     }
 
     fn grid_idx<I: std::convert::TryInto<usize>>(&self, x: I, y: I) -> Option<usize> {

examples/invaders/simple-invaders/Cargo.toml

@@ -5,7 +5,7 @@ authors = ["Jay Oster <[email protected]>"]
 edition = "2018"
 
 [dependencies]
-line_drawing = "1.0"
+clipline = "0.1.1"
 pcx = "0.2"
 randomize = "3.0"
 

examples/invaders/simple-invaders/src/sprites.rs

@@ -3,9 +3,7 @@ use crate::TIME_STEP;
 use crate::{Point, HEIGHT, WIDTH};
 use alloc::rc::Rc;
 use alloc::vec::Vec;
-use core::cmp::min;
 use core::time::Duration;
-use line_drawing::Bresenham;
 
 // This is the type stored in the `Assets` hash map
 pub(crate) type CachedSprite = (usize, usize, Rc<[u8]>);
@@ -215,17 +213,17 @@ where
 }
 
 /// Draw a line to the pixel buffer using Bresenham's algorithm.
-pub(crate) fn line(screen: &mut [u8], p1: &Point, p2: &Point, color: [u8; 4]) {
-    let p1 = (p1.x as i64, p1.y as i64);
-    let p2 = (p2.x as i64, p2.y as i64);
-
-    for (x, y) in Bresenham::new(p1, p2) {
-        let x = min(x as usize, WIDTH - 1);
-        let y = min(y as usize, HEIGHT - 1);
+pub(crate) fn line(screen: &mut [u8], p1: &Point, p2: &Point, color: [u8; 4]) -> Option<()> {
+    let p1 = (p1.x as isize, p1.y as isize);
+    let p2 = (p2.x as isize, p2.y as isize);
+    let clip_max = (WIDTH as isize - 1, HEIGHT as isize - 1);
+    for (x, y) in clipline::Clipline::new((p1, p2), ((0, 0), clip_max))? {
+        let (x, y) = (x as usize, y as usize);
         let i = x * 4 + y * WIDTH * 4;
 
         screen[i..i + 4].copy_from_slice(&color);
     }
+    Some(())
 }
 
 /// Draw a rectangle to the pixel buffer using two points in opposite corners.