qmk/qmk_firmware

[Bug] UCIS input not cleared after resolving mnemonic

Aperta

#22.438 aperta il 10 nov 2023

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)C (43.867 fork)batch import
bughelp wanted

Metriche repository

Star
 (20.368 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Describe the Bug

After resolving a unicode mnemonic, successfully or not, the input buffer should be cleared. In the current code the old input is kept in the buffer, potentially leading to incorrect resolution of future inputs.

Ex. enter the mnemonic "alpha", input is now "alpha". If you then enter the mnemonic "beta", input contains "betaa", which is incorrect.

I suggest using memset to zero out the input buffer at the end of ucis_finish.

Am I missing something important?

Keyboard Used

No response

Link to product page (if applicable)

No response

Operating System

No response

qmk doctor Output

No response

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

No response

Guida contributor

Direzione di ricerca
Esamina la funzione ucis finish nel codebase QMK per comprendere il comportamento attuale. Aggiungi memset per cancellare il buffer di input dopo la risoluzione del mnemonico. Assicurati che nessun'altra parte del codice faccia affidamento sulla conservazione del buffer. Testa con più mnemonici in sequenza.
Tech stack
c
Dominio
embedded iot
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CGitQMK
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70