Merge 81fbe89193 into 5d154f817a

src/Arduboy2.cpp

@@ -287,13 +287,25 @@ void Arduboy2Base::clear()
 
 void Arduboy2Base::drawPixel(int16_t x, int16_t y, uint8_t color)
 {
-  #ifdef PIXEL_SAFE_MODE
+  drawPixelMaybe(int16_t x, int16_t y, uint8_t color);
-  if (x < 0 || x > (WIDTH-1) || y < 0 || y > (HEIGHT-1))
+}
+
+// Draw a pixel, tell me whether it has been done.
+// The intent for inlining is to let drawPixel strip away the return part
+// and make drawPixelRaw a tail call.
+inline bool drawPixelMaybe(int16_t x, int16_t y, uint8_t color)
 {
-    return;
+  #ifdef PIXEL_SAFE_MODE
+  if (!validPixel(x, y))
+  {
+    return false;
   }
   #endif
+  drawPixelRaw(x, y, color);
+  return true;
+}
 
+void Arduboy2Base::drawPixelRaw(int16_t x, int16_t y, uint8_t color)
   uint16_t row_offset;
   uint8_t bit;
 
@@ -484,6 +496,13 @@ void Arduboy2Base::fillCircleHelper
 void Arduboy2Base::drawLine
 (int16_t x0, int16_t y0, int16_t x1, int16_t y1, uint8_t color)
 {
+  #ifdef PIXEL_SAFE_MODE
+  // No need to draw
+  if (!(validPixel(x0, y0) || validPixel(x1, y1))) {
+    return;
+  }
+  #endif
+
   // bresenham's algorithm - thx wikpedia
   bool steep = abs(y1 - y0) > abs(x1 - x0);
   if (steep) {
@@ -503,6 +522,9 @@ void Arduboy2Base::drawLine
   int16_t err = dx / 2;
   int8_t ystep;
   
+  bool initial_validity = steep ? validPixel(y0, x0) : validPixel(x0, y0);
+  bool valid;
+
   if (y0 < y1)
   {
     ystep = 1;
@@ -516,11 +538,16 @@ void Arduboy2Base::drawLine
   {
     if (steep)
     {
-      drawPixel(y0, x0, color);
+      valid = drawPixel(y0, x0, color);
     }
     else
     {
-      drawPixel(x0, y0, color);
+      valid = drawPixel(x0, y0, color);
+    }
+
+    // We have reached the end of valid pixels.
+    if (initial_validity && !valid) {
+      break;
     }
 
     err -= dy;
@@ -544,10 +571,14 @@ void Arduboy2Base::drawRect
 void Arduboy2Base::drawFastVLine
 (int16_t x, int16_t y, uint8_t h, uint8_t color)
 {
+  // Do x bounds checks
+  if (x < 0 || x >= WIDTH)
+    return;
+
   int end = y+h;
   for (int a = max(0,y); a < min(end,HEIGHT); a++)
   {
-    drawPixel(x,a,color);
+    drawPixelRaw(x,a,color);
   }
 }
 

src/Arduboy2.h

@@ -480,6 +480,20 @@ class Arduboy2Base : public Arduboy2Core
    */
   void display(bool clear);
 
+  /** \brief
+   * Tells whether a given coordinate points to a valid pixel.
+   * 
+   * \param x The X coordinate of the pixel.
+   * \param y The Y coordinate of the pixel.
+   * 
+   * \details
+   * This function returns true if x is in the interval [0, WIDTH) and y is in
+   * the interval [0, HEIGHT).
+   */
+  static constexpr bool validPixel(int16_t x, int16_t y) {
+    return !(x < 0 || x > (WIDTH-1) || y < 0 || y > (HEIGHT-1));
+  }
+
   /** \brief
    * Set a single pixel in the display buffer to the specified color.
    *
@@ -494,6 +508,10 @@ class Arduboy2Base : public Arduboy2Core
    */
   static void drawPixel(int16_t x, int16_t y, uint8_t color = WHITE);
 
+  // Draw a pixel without checking.
+  // (Not officially part of the API)
+  void drawPixelRaw(int16_t x, int16_t y, uint8_t color);
+
   /** \brief
    * Returns the state of the given pixel in the screen buffer.
    *