search

MEGA65 / mega65-core

MEGA65 FPGA core
Other
238 stars 84 forks source link

Typing event queue: repeat is triggering too early? #738

Closed dansanderson closed 9 months ago

dansanderson commented 9 months ago

Test Environment (required) You can use MEGA65INFO to retrieve this.

Describe the bug I'm occasionally seeing multiple strikes of a key press using the new keyboard scanner based on the typing event queue. At first I thought maybe our debounce timing needed tweaking, but as I experimented more I realized that when it happened, it appeared to have the frequency of the hardware-based key repeat logic, as if the key was in "repeat" mode on first press. I wonder if the repeat delay isn't getting reset correctly under certain circumstances, and enables repeat mode even when I have moved on to the next key.

gardners commented 9 months ago

That sounds like a plausbile explanation. Alternatively, it could be that the initial delay is not being reset, and it is using whatever fraction of it remains until the next timeout of it, or alternatively that it is using the repeat interval instead.