qmk/qmk_firmware

[Bug] UCIS input not cleared after resolving mnemonic

Open

#22 438 ouverte le 10 nov. 2023

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)C (43 867 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (20 368 stars)
Métriques de merge PR
 (Merge moyen 20j 9h) (27 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez la fonction ucis finish dans la base de code QMK pour comprendre le comportement actuel. Ajoutez memset pour effacer le tampon d'entrée après la résolution du mnémonique. Assurez vous qu'aucune autre partie du code ne dépend de la conservation du tampon. Testez avec plusieurs mnémoniques en séquence.
Stack technique
c
Domaine
embedded iot
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CGitQMK
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70