Page MenuHomePhabricator

Ignores space in escape sequences
Open, Incoming QueuePublic

Description

ECMA-48's

8.3.121 SL - SCROLL LEFT
Representation: CSI Pn 02/00 04/00

is incorrectly interpreted as

8.3.64 ICH - INSERT CHARACTER
Representation: CSI Pn 04/00

Similarly,

8.3.130 SPD - SELECT PRESENTATION DIRECTIONS
Representation: CSI Ps1;Ps2  02/00 05/03

is misinterpreted as

8.3.147 SU - SCROLL UP
Representation: CSI Pn 05/03

and so on for presumably many more.

Probably Terminology doesn't implement any of these escape sequences with a space inside, otherwise this bug would have been noticed earlier. Still, ideally the parser should be changed so that these escape sequences are silently ignored, rather than doing something else.

Also note that both of these

\e[ 31m
\e[31 m

switch to red foreground, this isn't expected either.

\e[3 1m

switches to italic and bold, as if that space was a semicolon instead.

egmont created this task.Nov 18 2018, 9:24 AM
egmont renamed this task from Ignores space in escape sequences (#1153) to Ignores space in escape sequences.Nov 18 2018, 9:25 AM
billiob claimed this task.Mon, Jan 21, 1:21 PM