domiluci / liquidcrystal_nkc Goto Github PK

I am using your library on a LCD2004 with PCF8574 I2C connection and I am getting an error during compiling. It seems like its an error in the library and not my code but I am not sure. The processor is an Arduino Nano ESP32 board.

I am using your full demo script with no additional code as a function test and tried to compile it and it gives me the following error:

...Arduino\libraries\LiquidCrystal_NKC\src\LiquidCrystal_NKC.cpp:428:41: note: deduced conflicting types for parameter 'const _Tp' ('unsigned int' and 'int')
len = min(strlen(inputChar), MAX_STRLEN);
^

Official Bug Report for LiquidCrystal NKC v0.21.309

Per the LiquidCrystal NKC discussion thread on the official Arduino Forum, a few bugs and issues have been presented in v0.21.309 that I am self-reporting for clarity and accountability, and that I have every intention of resolving by v0.21.316 (Nov. 12th, 2021).

Items with no check marks are issues that have not yet been resolved. Items with check marks are issue that have been resolved and will be included as official fixes in the next weekly release.

MAJOR BUGS

• Using the I2C_SLOW constant to set the I2C speed to 10kHz breaks functionality. New value suggestion is 50kHz.

OTHER ISSUES

• The delayMicroseconds(I2CDELAY) line in the I2C command functions is now only after Wire.endTransmission()
• Some of the delayMicroseconds(RS232DELAY) uses in the RS232 command functions may break functionality.

FORMATTING

• There is an issue with the indentation and formatting of LiquidCrystal_NKC.cpp & LiquidCrystal_NKC.h.

LONG-TERM CHANGES

1. The biggest goal of LiquidCrystal NKC is universal compatibility. Therefore, LC NKC will be made LCD API 1.0 compliant.
2. The use of init() will be deprecated, yet remain supported. The suggested function going forward will be begin().
3. The setBacklight() function will be altered to comply with the LCD API 1.0 standard of a non-binary dimming level. To simply set the backlight ON or OFF, use LiquidCrystal compatible backlight() and noBacklight().
4. Support for more major boards & board-specific pins that auto-update based on the board selected in the IDE.

If there are any other issues and bugs in v0.21.309, and/or suggestions for improvement that you would like to include, please make them known in this thread below. If you're having an issue with another version of LiquidCrystal NKC, please find that version's official bug report thread, or start a new thread in the Issues section of this repository