emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[debbugs-tracker] bug#31105: closed ([PATCH] Restart and reset ANSI esca


From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#31105: closed ([PATCH] Restart and reset ANSI escapes on each line for compatibility with less -R)
Date: Mon, 31 Dec 2018 06:26:02 +0000

Your message dated Sun, 30 Dec 2018 22:25:35 -0800
with message-id <address@hidden>
and subject line Re: [bug-diffutils] bug#31105: bug#31105: [PATCH] Restart and 
reset ANSI escapes on each line for compatibility with less -R
has caused the debbugs.gnu.org bug report #31105,
regarding [PATCH] Restart and reset ANSI escapes on each line for compatibility 
with less -R
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
31105: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=31105
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: [PATCH] Restart and reset ANSI escapes on each line for compatibility with less -R Date: Sun, 08 Apr 2018 19:59:31 -0400 User-agent: Internet Messaging Program (IMP) H3 (4.3)
Hello!

GNU less can display ANSI-colored text with the -R flag, but this support has some limitations. One of them is that if an escape sequence starts on one line and ends on a different line, only the first line will be colored in less.

As a result, when diff creates colored output with multi-line deletes or adds, less will only color the first line.

I've attached a patch to reset ANSI color to the default at the end of every line and restart it at the beginning of the next. It patches normal and context mode. Side-by-side already worked in my testing.

I hope it's useful to you. Please let me know if there are changes you would like made before you can accept it.

--Dennis Lambe Jr.

Attachment: 99-color-escapes-every-line.patch
Description: Text Data


--- End Message ---
--- Begin Message --- Subject: Re: [bug-diffutils] bug#31105: bug#31105: [PATCH] Restart and reset ANSI escapes on each line for compatibility with less -R Date: Sun, 30 Dec 2018 22:25:35 -0800
On Sun, Dec 30, 2018 at 8:18 PM Dennis Lambe Jr. <address@hidden> wrote:
> Quoting Jim Meyering <address@hidden>:
> > Here's the result. Please review carefully and let me know if you
> > would like to change anything. I won't push until after I've heard
> > from you.
>
> Looks great to me -- thanks!

Thanks. With that, I've pushed it and am marking this as "done".


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]