uint8_t used instead of uint16_t incorrectly in GFX library

[hchahine]

Within the ILI9341_GFX library, you've declared char and text functions as follows:

void ILI9341_Draw_Char(char Character, uint8_t X, uint8_t Y, uint16_t Colour, uint16_t Size, uint16_t Background_Colour);

void ILI9341_Draw_Text(const char* Text, uint8_t X, uint8_t Y, uint16_t Colour, uint16_t Size, uint16_t Background_Colour);

The X and Y variables can overflow and limit writing of text or characters all the way to the edge of the screen (especially in Horizontal screen mode). E.g. Max X or Y is only 256 while 240x320 screen can go past this to 320.

While this inadvertently allows text to rollover, which may be desirable in some cases, it limits performance of the library. There are better ways to write rollover feature if so desired.