xtermjs/xterm.js

Infinite loop when drawing large ligatures to char atlas

Open

#4362 aperta il 2 gen 2023

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (1574 fork)batch import
area/addon/ligaturesarea/addon/webglhelp wantedtype/bug

Metriche repository

Star
 (16.196 star)
Metriche merge PR
 (Merge medio 4g 3h) (81 PR mergiate in 30 g)

Descrizione

some fonts provide extremely long ligatures, e.g. Fira Code for =================================================== The char atlas code tries to find a row to fit it but fails and gets stuck in an infinite loop https://github.com/xtermjs/xterm.js/blob/fe8981766769c19ab1903dffa19f1a64ad83da13/src/browser/renderer/shared/TextureAtlas.ts#L792-L799 ^Here we're checking if there's room on the current row and creating a new one if not.

Details

  • Browser and browser version: chrome 108
  • OS version: macOS 12
  • xterm.js version: 5.1.0 (also on latest master branch)

Steps to reproduce

  1. Install Fira Code font, Run the xterm demo and enable ligatures
  2. start typing =
  3. Keep looking at the char atlas drawn below, once the glyph is almost the same width as char atlas, typing the next = will freeze everything.

Guida contributor

Direzione di ricerca
Esamina il ciclo alle righe 792-799 di TextureAtlas.ts e implementa una protezione per rilevare quando una nuova riga non può ospitare la legatura, prevenendo il ciclo infinito.
Tech stack
typescript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70