Description
Both TpagerKeyboard and CardputerKeyboard (Adv) currently call xQueueSend with a 50ms timeout but do not check the return value. If the queue is full and the timeout expires, keypresses will be silently dropped, leading to a poor user experience during rapid typing.
Affected Files
Boards/LilygoTLoraPager/Source/devices/TpagerKeyboard.cpp (line 93)Boards/M5stackCardputerAdv/Source/devices/CardputerKeyboard.cpp
Expected Behavior
The code should check the return value of xQueueSend and handle failures appropriately (e.g., log a warning when keypresses are dropped).
References
Description
Both TpagerKeyboard and CardputerKeyboard (Adv) currently call
xQueueSendwith a 50ms timeout but do not check the return value. If the queue is full and the timeout expires, keypresses will be silently dropped, leading to a poor user experience during rapid typing.Affected Files
Boards/LilygoTLoraPager/Source/devices/TpagerKeyboard.cpp(line 93)Boards/M5stackCardputerAdv/Source/devices/CardputerKeyboard.cppExpected Behavior
The code should check the return value of
xQueueSendand handle failures appropriately (e.g., log a warning when keypresses are dropped).References