Re: bogl-term reverse video patch, please review
Chuan-kai Lin <cklin@oink.cc.ntu.edu.tw> writes:
> On Fri, Jan 19, 2001 at 11:43:52AM +0800, ha shao wrote:
> > I like your patch. I think it is more generic than the one I sent
> > to the list a couple days ago. I just think the sgr0 should be
> > \E[0m as like most other terminfo. sgr0 reset the terminal into
> > default setting. It should reset underline, reverse video, forground,
> > background. \E[8m in ANSI escape sequence means invisible.
>
> Actually I did not know that you were working on the same issue;
> otherwise I would find myself other problems to tackle :) Changing the
> sgr0 code should be a good idea. I am not familiar with terminfo code
> conventions, and a quick glance showed that linux, xterm, and rxvt all
> use different codes for sgr0. So I just picked one at random.
Ouch. I just committed ha shao's patch. Chuan, please decide which
is better and be sure what is in CVS is in a good state.
That's what you get for no-one committing fixes and me not knowing any
better and getting lagged on emails. I just pick whatever is the
first patch to come along.
--
.....Adam Di Carlo....adam@onShore.com.....<URL:http://www.onShore.com/>
Reply to: