Re: [dev] [st] [PATCH] More compatible with xterm

2013-10-01 Thread Roberto E. Vargas Caballero
> > It look like problem not related to save/restore cursor position. This > test have strange sequence: ESC[m ESC(B ESC)B > What this sequence should do? ESC[m = SGR. Parameter=0 restore default graphic rendering values ESC(B = SCS. Selects USASCII in primary charset (G0) ESC)B = SCS. Selects US

Re: [dev] [st] [PATCH] More compatible with xterm

2013-09-30 Thread Mihail Zenkov
2013/9/30, Roberto E. Vargas Caballero : > I meant: 2. Test of screen features (last test of the section: SAVE/RESTORE > CURSOR feature). Be careful of running this test without 24x80.24. It look like problem not related to save/restore cursor position. This test have strange sequence: ESC[m ESC(B

Re: [dev] [st] [PATCH] More compatible with xterm

2013-09-30 Thread Roberto E. Vargas Caballero
> > I meant: 2. Test of screen features (last test of the section: SAVE/RESTORE > CURSOR feature). Be careful of running this test without 24x80.24. > > Sorry, s/without/with/ -- Roberto E. Vargas Caballero k...@shike2.com http://www.shike2.com

Re: [dev] [st] [PATCH] More compatible with xterm

2013-09-30 Thread Roberto E. Vargas Caballero
> Are you mean test this: 11. Test non-VT100 (e.g., VT220, XTERM) > terminals -> 8. Test XTERM special features -> 7. Alternate-Screen > features (xterm) ? With my patches st pass this test. I meant: 2. Test of screen features (last test of the section: SAVE/RESTORE CURSOR feature). Be careful of

Re: [dev] [st] [PATCH] More compatible with xterm

2013-09-30 Thread Mihail Zenkov
2013/9/30, Roberto E. Vargas Caballero : >> >> I check this. To pass this test we also should report cursor position. > > Again, your patch is good, but I have tested vttest and it > continues failing. I am going to apply your patches in a week > (I usually wait a week before appling any patch, so

Re: [dev] [st] [PATCH] More compatible with xterm

2013-09-30 Thread Roberto E. Vargas Caballero
> > I check this. To pass this test we also should report cursor position. Again, your patch is good, but I have tested vttest and it continues failing. I am going to apply your patches in a week (I usually wait a week before appling any patch, so anyone can look it and catch a bug), but I think

Re: [dev] [st] [PATCH] More compatible with xterm

2013-09-29 Thread Mihail Zenkov
2013/9/29, Roberto E. Vargas Caballero : > On Sat, Sep 28, 2013 at 08:47:04PM +, Mihail Zenkov wrote: >> I have problem in mc with incorrect cursor position in alt screen. I >> check xterm and found two difference. >> >> 1. xterm store two cursor position - first for main screen and second >> f

Re: [dev] [st] [PATCH] More compatible with xterm

2013-09-29 Thread Roberto E. Vargas Caballero
On Sat, Sep 28, 2013 at 08:47:04PM +, Mihail Zenkov wrote: > I have problem in mc with incorrect cursor position in alt screen. I > check xterm and found two difference. > > 1. xterm store two cursor position - first for main screen and second > for alt screen. > Explaining: http://invisible-i

[dev] [st] [PATCH] More compatible with xterm

2013-09-28 Thread Mihail Zenkov
I have problem in mc with incorrect cursor position in alt screen. I check xterm and found two difference. 1. xterm store two cursor position - first for main screen and second for alt screen. Explaining: http://invisible-island.net/xterm/xterm.log.html#xterm_90 2. Escape sequence [?1049l swap sc