Code Review requested changes

_textsize_x -> _textSizeX  (ditto for y)
setTextSize() -> textSize()

scaledBitmap -> bitmap with the extra parameters with defaults

Author: KurtE

src/ArduinoGraphics.cpp

@@ -27,8 +27,8 @@ ArduinoGraphics::ArduinoGraphics(int width, int height) :
   _width(width),
   _height(height),
   _font(NULL),
-  _textsize_x(1),
-  _textsize_y(1)
+  _textSizeX(1),
+  _textSizeY(1)
 {
 }
 
@@ -243,7 +243,7 @@ void ArduinoGraphics::text(const char* str, int x, int y)
     uint8_t const c = (uint8_t)*str++;
 
     if (c == '\n') {
-      y += _font->height * _textsize_y;
+      y += _font->height * _textSizeY;
     } else if (c == '\r') {
       x = 0;
     } else if (c == 0xc2 || c == 0xc3) {
@@ -256,10 +256,10 @@ void ArduinoGraphics::text(const char* str, int x, int y)
       }
 
       if (b) {
-        scaledBitmap(b, x, y, _font->width, _font->height, _textsize_x, _textsize_y);
+        bitmap(b, x, y, _font->width, _font->height, _textSizeX, _textSizeY);
       }
 
-      x += _font->width * _textsize_x;
+      x += _font->width * _textSizeX;
     }
   }
 }
@@ -271,28 +271,22 @@ void ArduinoGraphics::textFont(const Font& which)
 
 int ArduinoGraphics::textFontWidth() const
 {
-  return (_font ? _font->width * _textsize_x : 0);
+  return (_font ? _font->width * _textSizeX : 0);
 }
 
 int ArduinoGraphics::textFontHeight() const
 {
-  return (_font ? _font->height* _textsize_y : 0);
+  return (_font ? _font->height* _textSizeY : 0);
 }
 
-void ArduinoGraphics::setTextSize(uint8_t sx, uint8_t sy)
+void ArduinoGraphics::textSize(uint8_t sx, uint8_t sy)
 {
-  _textsize_x = (sx > 0)? sx : 1;
-  _textsize_y = (sy > 0)? sy : 1;
+  _textSizeX = (sx > 0)? sx : 1;
+  _textSizeY = (sy > 0)? sy : 1;
 }
 
 
-void ArduinoGraphics::bitmap(const uint8_t* data, int x, int y, int width, int height)
-{
-  // forward to scaled version
-  scaledBitmap(data, x, y, width, height, 1, 1);
-}
-
-void ArduinoGraphics::scaledBitmap(const uint8_t* data, int x, int y, int w, int h, uint8_t scale_x, uint8_t scale_y) {
+void ArduinoGraphics::bitmap(const uint8_t* data, int x, int y, int w, int h, uint8_t scale_x, uint8_t scale_y) {
   if (!_stroke || !scale_x || !scale_y) {
     return;
   }
@@ -380,7 +374,7 @@ void ArduinoGraphics::image(const Image& img, int x, int y)
 
 void ArduinoGraphics::image(const Image& img, int x, int y, int width, int height)
 {
-  if (!img || ((x + width) < 0) || ((y + height) < 0) || (x > _width) || (y > height)) {
+  if (!img || ((x + width) < 0) || ((y + height) < 0) || (x > _width) || (y > _height)) {
     // offscreen
     return;
   }
@@ -459,7 +453,7 @@ void ArduinoGraphics::endText(int scrollDirection)
       beginDraw();
       int const text_x = _textX - i;
       text(_textBuffer, text_x, _textY);
-      scaledBitmap(_font->data[0x20], text_x - 1, _textY, 1, _font->height, _textsize_x, _textsize_y);
+      bitmap(_font->data[0x20], text_x - 1, _textY, 1, _font->height, _textSizeX, _textSizeY);
       endDraw();
 
       delay(_textScrollSpeed);
@@ -471,7 +465,7 @@ void ArduinoGraphics::endText(int scrollDirection)
       beginDraw();
       int const text_x = _textX - (scrollLength - i - 1);
       text(_textBuffer, text_x, _textY);
-      scaledBitmap(_font->data[0x20], text_x - 1, _textY, 1, _font->height, _textsize_x, _textsize_y);
+      bitmap(_font->data[0x20], text_x - 1, _textY, 1, _font->height, _textSizeX, _textSizeY);
       endDraw();
 
       delay(_textScrollSpeed);
@@ -483,7 +477,7 @@ void ArduinoGraphics::endText(int scrollDirection)
       beginDraw();
       int const text_y = _textY - i;
       text(_textBuffer, _textX, text_y);
-      scaledBitmap(_font->data[0x20], _textX, text_y - 1, _font->width, 1, _textsize_x, _textsize_y);
+      bitmap(_font->data[0x20], _textX, text_y - 1, _font->width, 1, _textSizeX, _textSizeY);
       endDraw();
 
       delay(_textScrollSpeed);
@@ -495,7 +489,7 @@ void ArduinoGraphics::endText(int scrollDirection)
       beginDraw();
       int const text_y = _textY - (scrollLength - i - 1);
       text(_textBuffer, _textX, text_y);
-      scaledBitmap(_font->data[0x20], _textX, text_y - 1, _font->width, 1, _textsize_x, _textsize_y);
+      bitmap(_font->data[0x20], _textX, text_y - 1, _font->width, 1, _textSizeX, _textSizeY);
       endDraw();
 
       delay(_textScrollSpeed);

src/ArduinoGraphics.h

@@ -70,8 +70,8 @@ class ArduinoGraphics : public Print {
   virtual void text(const char* str, int x = 0, int y = 0);
   virtual void text(const String& str, int x = 0, int y = 0) { text(str.c_str(), x, y); }
   virtual void textFont(const Font& which);
-  virtual void setTextSize(uint8_t s) {setTextSize(s, s);}
-  virtual void setTextSize(uint8_t sx, uint8_t sy);
+  virtual void textSize(uint8_t s) {textSize(s, s);}
+  virtual void textSize(uint8_t sx, uint8_t sy);
 
   virtual int textFontWidth() const;
   virtual int textFontHeight() const;
@@ -93,9 +93,8 @@ class ArduinoGraphics : public Print {
   virtual void textScrollSpeed(unsigned long speed = 150);
 
 protected:
-  virtual void bitmap(const uint8_t* data, int x, int y, int width, int height);
-  virtual void scaledBitmap(const uint8_t* data, int x, int y, int width, int height, 
-                            uint8_t scale_x, uint8_t);
+  virtual void bitmap(const uint8_t* data, int x, int y, int w, int h, uint8_t scale_x = 1, 
+                      uint8_t scale_y = 1);
   virtual void imageRGB(const Image& img, int x, int y, int width, int height);
   virtual void imageRGB24(const Image& img, int x, int y, int width, int height);
   virtual void imageRGB16(const Image& img, int x, int y, int width, int height);
@@ -118,8 +117,8 @@ class ArduinoGraphics : public Print {
   uint8_t _textR, _textG, _textB;
   int _textX;
   int _textY;
-  uint8_t _textsize_x;
-  uint8_t _textsize_y;
+  uint8_t _textSizeX;
+  uint8_t _textSizeY;
   unsigned long _textScrollSpeed;
 };