Bug#200857: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=200857
Debian Bug report logs - #200857 xterm fails to honour dtterm sequence ESC[21t ... allowWindowOps (class AllowWindowOps) Specifies whether extended windowcontrol sequences (as used in dtterm) for should be allowed. ... From: Vivek [EMAIL PROTECTED] ... Ok then: assuming this defaults/will default to false, there's no real need to stomp on the sequence handling completely, then, right? That's my viewpoint - if Brendan adds a false to the system app-defaults file, it would still allow a user to override it and get the features. Making it return an empty string would be incorrect behavior - if it's not implemented, it should be parsed ignored. -- Thomas E. Dickey [EMAIL PROTECTED] http://invisible-island.net ftp://invisible-island.net -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#200857: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=200857
Debian Bug report logs - #200857 xterm fails to honour dtterm sequence ESC[21t ... allowWindowOps (class AllowWindowOps) Specifies whether extended windowcontrol sequences (as used in dtterm) for should be allowed. ... From: Vivek [EMAIL PROTECTED] ... Ok then: assuming this defaults/will default to false, there's no real need to stomp on the sequence handling completely, then, right? That's my viewpoint - if Brendan adds a false to the system app-defaults file, it would still allow a user to override it and get the features. Making it return an empty string would be incorrect behavior - if it's not implemented, it should be parsed ignored. -- Thomas E. Dickey [EMAIL PROTECTED] http://invisible-island.net ftp://invisible-island.net
Bug#200857: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=200857
Debian Bug report logs - #200857 xterm fails to honour dtterm sequence ESC[21t ... I think it might be better to do something like this [see patch]: The iscntrl() test can be replaced easily if more/less paranoia is required - it's not really fair to expect apps to figure out the escape sequence is just gone, as afaict there's no way to ask if a term supports many of them other than checking TERM and going by what you know/expect that terminal to support. not exactly - the label doesn't have control characters - see this chunk in misc.c: default: if (ansi_table[CharOf(*cp)] != CASE_PRINT) return; } The complaint is that someone could be misled into pressing return after a bogus title was sent to the host, e.g., as part of cat'ing a core-dump to the terminal. (Why anyone would want to do this is another story). Or doing tail -f on a so-called logfile (no different ;-). This is in patch #174 (would allow someone to use these features if they're inclined to read the manpage): allowWindowOps (class AllowWindowOps) Specifies whether extended windowcontrol sequences (as used in dtterm) for should be allowed. -- Thomas E. Dickey [EMAIL PROTECTED] http://invisible-island.net ftp://invisible-island.net -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Bug#200857: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=200857
Debian Bug report logs - #200857 xterm fails to honour dtterm sequence ESC[21t ... I think it might be better to do something like this [see patch]: The iscntrl() test can be replaced easily if more/less paranoia is required - it's not really fair to expect apps to figure out the escape sequence is just gone, as afaict there's no way to ask if a term supports many of them other than checking TERM and going by what you know/expect that terminal to support. not exactly - the label doesn't have control characters - see this chunk in misc.c: default: if (ansi_table[CharOf(*cp)] != CASE_PRINT) return; } The complaint is that someone could be misled into pressing return after a bogus title was sent to the host, e.g., as part of cat'ing a core-dump to the terminal. (Why anyone would want to do this is another story). Or doing tail -f on a so-called logfile (no different ;-). This is in patch #174 (would allow someone to use these features if they're inclined to read the manpage): allowWindowOps (class AllowWindowOps) Specifies whether extended windowcontrol sequences (as used in dtterm) for should be allowed. -- Thomas E. Dickey [EMAIL PROTECTED] http://invisible-island.net ftp://invisible-island.net
