Blightmud / Blightmud / 22755561323

74%

Build Type

push

github

Committed by web-flow

Commit Message

Fix wrap_line() to handle all ANSI escape sequence types (#1352)

The escape sequence parser in wrap_line() only recognized SGR sequences
(\x1b[...m) and would consume all characters until the next 'm' when
encountering any other escape type. This caused OSC 8 hyperlinks (emitted
by pulldown-cmark-mdcat for markdown links), CSI non-SGR sequences, DCS,
and APC sequences to eat visible text content.

Replace the simple in_escape boolean with a proper state machine that
handles CSI (terminated by 0x40-0x7E), OSC (terminated by BEL or ST),
DCS/SOS/PM/APC (terminated by ST), and simple two-byte ESC sequences.

Co-authored-by: daiimus <daiimus@gmail.com>

Coverage Stats

84 of 113 new or added lines in 1 file covered. (74.34%)

1 existing line in 1 file now uncovered.

8857 of 11242 relevant lines covered (78.78%)

764443.13 hits per line