extmod/modframebuf: optimize fill_rect subroutine call

kamikaze · kamikaze · commit e45035db5c29 · 2017-01-08T20:16:17.000+02:00
diff --git a/extmod/modframebuf.c b/extmod/modframebuf.c
@@ -115,7 +115,7 @@ static inline uint32_t getpixel(const mp_obj_framebuf_t *fb, int x, int y) {
 }
 
 STATIC void fill_rect(const mp_obj_framebuf_t *fb, int x, int y, int w, int h, uint32_t col) {
-    if (x + w <= 0 || y + h <= 0 || y >= fb->height || x >= fb->width) {
+    if (h < 1 || w < 1 || x + w <= 0 || y + h <= 0 || y >= fb->height || x >= fb->width) {
         // No operation needed.
         return;
     }

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ static inline uint32_t getpixel(const mp_obj_framebuf_t *fb, int x, int y) {`
`115`	`115`	`}`
`116`	`116`
`117`	`117`	`STATIC void fill_rect(const mp_obj_framebuf_t *fb, int x, int y, int w, int h, uint32_t col) {`
`118`		`- if (x + w <= 0 \|\| y + h <= 0 \|\| y >= fb->height \|\| x >= fb->width) {`
	`118`	`+ if (h < 1 \|\| w < 1 \|\| x + w <= 0 \|\| y + h <= 0 \|\| y >= fb->height \|\| x >= fb->width) {`
`119`	`119`	`// No operation needed.`
`120`	`120`	`return;`
`121`	`121`	`}`