Remove the toolkit's _ev_flag modifier shim once EigenOS ships bool modifiers (EigenOS#15)

Aperta
#2,192 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
53/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
c
Ambito
tooling

Direzione di ricerca

Read lib/ui.eigs:32-38 and inspect the _ev_flag call sites to see which modifier flags are normalized. Confirm EigenOS#15 has shipped and consumers use the bool emitting version before removing the shim. Add or update a toolkit test for boolean modifier flags on the SDL path; done means the shim is removed, call sites read the event fields directly, and the test passes.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

area:gfx enhancement

lib/ui.eigs:32-38 (_ev_flag, #1637) normalizes event modifier flags because EigenOS reported shift/ctrl/alt as numbers while ext_gfx.c reports bools. EigenOS#15 (PR EigenOS#19) makes EigenOS emit bools, matching ext_gfx.c, so after that ships the shim is dead weight.

Done when

  • Once EigenOS with bool modifiers is the version consumers run, _ev_flag is removed, its call sites read ev.shift and the others directly, and a toolkit test pins that modifier flags are bools on the SDL path.

— Claude (orchestrator, Claude Code)

Lingua principale
C
Stelle
3
Fork
7
Merge medio
6h 32m
PR unite (30g)
285

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di InauguralSystems/EigenScript

Tutte le issue di InauguralSystems/EigenScript

Issue simili

Altre issue su C

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.