Only two resolutions are supported, 480x272 and 800x480. As currently structured, if the user gives begin() a number other than these 2 legal value, this library can do wrong things. This change checks for a wrong value and returns with false.
A couple other resolution checks are changed if/else, which makes the compiler much happier if warnings are enabled.
I tested on a Teensy 3.2 with the Adafruit RA8875 breakout and a 800x480 display.
ladyada
commented
7 years ago
This is a very minor fix for compiler warnings.
Only two resolutions are supported, 480x272 and 800x480. As currently structured, if the user gives begin() a number other than these 2 legal value, this library can do wrong things. This change checks for a wrong value and returns with false.
A couple other resolution checks are changed if/else, which makes the compiler much happier if warnings are enabled.
I tested on a Teensy 3.2 with the Adafruit RA8875 breakout and a 800x480 display.