Open orzel opened 3 years ago
@orzel Sorry for delay. I do not have access to this screen anymore therefore I'm unable to maintain this. Anyway I have merged all pending PRs
Let me know if this fix the issue. so I can make a release
I just added a pull request with the needed change. I tested it on my hardware with Arduino Uno and TFT-shield.
Hopefully this issue can be closed ;-)
Arduino board: I use the arduino mega 2560 but that's unrelated.
Arduino IDE version (found in Arduino -> About Arduino menu): 1.8.13 VERSION HERE**
List the steps to reproduce the problem below (if possible attach a sketch or copy the sketch code in too): load the graphictest from file>examples>SPFD5408 TFT Library
I have several errors (see end of post). The first and most obvious one is that read8inline is defined twice, and indeed, that can still be seen in github as of today: https://github.com/sadika9/TFTLCD-SPFD5408/blob/3980b7b17d0a8a822548dbcf7d09a92f2921d2f8/src/pin_magic.h#L174
The second definition is following the first, so that really look like a copy/paste problem. Other errors follow and prevent compilation. As i note that the code hasnn't been touched for 3 years and that pull request have been ignored for as long... Is this code still maintained ? If not, could you make it explicit either by a warning or by removing the repository ?
(it seems that one of the pull requests solves those errors)
/home/orzel/Arduino/libraries/SPFD5408_TFT_Library/src/TftSpfd5408.cpp: In member function ‘void TftSpfd5408::init()’: /home/orzel/Arduino/libraries/SPFD5408_TFT_Library/src/TftSpfd5408.cpp:115:3: error: ‘textsize’ was not declared in this scope; did you mean ‘textsize_x’? 115 | textsize = 1; | ^
~~~ | textsize_x In file included from /home/orzel/Arduino/libraries/SPFD5408_TFT_Library/src/TftSpfd5408.cpp:19: /home/orzel/Arduino/libraries/SPFD5408_TFT_Library/src/TftSpfd5408.cpp: In member function ‘void TftSpfd5408::begin(uint16_t)’: /home/orzel/Arduino/libraries/SPFD5408_TFT_Library/src/TftSpfd5408.cpp:283:43: warning: unsigned conversion from ‘int’ to ‘uint8_t’ {aka ‘unsigned char’} changes value from ‘11051’ to ‘43’ [-Woverflow] 283 | writeRegister16(ILI9341_VCOMCONTROL1, 0x2B2B); /home/orzel/Arduino/libraries/SPFD5408_TFT_Library/src/pin_magic.h:416:24: note: in definition of macro ‘writeRegister16inline’ 416 | hi = (d) >> 8; lo = (d); CD_DATA ; write8(hi); write8(lo); } | ^ /home/orzel/Arduino/libraries/SPFD5408_TFT_Library/src/TftSpfd5408.cpp:283:5: note: in expansion of macro ‘writeRegister16’ 283 | writeRegister16(ILI9341_VCOMCONTROL1, 0x2B2B); | ^~~~~~~