Bug#466789: Parsing of terminal version response too strict?

2008-02-25 Thread Thomas Dickey
On Sun, Feb 24, 2008 at 11:00:18PM +0100, Bram Moolenaar wrote: We don't know what it is. Could be a function key. I think every escape sequence is supposed to end in a letter. So we could use this loop: for (i = 2 + (tp[0] != CSI); i len !ASCII_ISALPHA(tp[i]); ++i)

Bug#466789: Parsing of terminal version response too strict?

2008-02-25 Thread Bram Moolenaar
Thomas - On Sun, Feb 24, 2008 at 11:00:18PM +0100, Bram Moolenaar wrote: We don't know what it is. Could be a function key. I think every escape sequence is supposed to end in a letter. So we could use this loop: for (i = 2 + (tp[0] != CSI); i len

Bug#466789: Parsing of terminal version response too strict?

2008-02-25 Thread Thomas Dickey
On Mon, Feb 25, 2008 at 08:35:26PM +0100, Bram Moolenaar wrote: Technically, they could also end with '{', '|' or '}', though I don't recall any terminals which do this. Good to know. I now made it: for (i = 2 + (tp[0] != CSI); i len

Bug#466789: Parsing of terminal version response too strict?

2008-02-24 Thread Bram Moolenaar
James - In term.c's check_termcode, there is code to handle the response to CSI0c (request terminal version) which is of the form CSI{x};{vers};{y}c. The format of {vers} isn't strictly defined but the current code assumes that the response will only be made up of digits, ';', and '.'.

Bug#466789: Parsing of terminal version response too strict?

2008-02-24 Thread James Vega
On Sun, Feb 24, 2008 at 04:47:21PM +0100, Bram Moolenaar wrote: James - In term.c's check_termcode, there is code to handle the response to CSI0c (request terminal version) which is of the form CSI{x};{vers};{y}c. The format of {vers} isn't strictly defined but the current code

Bug#466789: Parsing of terminal version response too strict?

2008-02-24 Thread Bram Moolenaar
James - In term.c's check_termcode, there is code to handle the response to CSI0c (request terminal version) which is of the form CSI{x};{vers};{y}c. The format of {vers} isn't strictly defined but the current code assumes that the response will only be made up of digits, ';',