GNU bug report logs - #31105
[PATCH] Restart and reset ANSI escapes on each line for compatibility with less -R

Previous Next

Full log

View this message in rfc822 format

From: "Dennis Lambe Jr." <malsyned <at> malsyned.net>
To: 31105 <at> debbugs.gnu.org
Subject: bug#31105: [bug-diffutils] bug#31105: [PATCH] Restart and reset ANSI escapes on each line for compatibility with less -R
Date: Mon, 09 Apr 2018 11:53:17 -0400

[Message part 1 (text/plain, inline)]

The previous patch had an error in the first hunk for normal.c. I  
forgot to remove an "if" line when I removed the following conditional  
line. Due to its placement before a "for" line that tested the same  
condition, it didn't affect the behavior of the patch, but it was  
unsightly. I've attached an updated patch.

Quoting "Dennis Lambe Jr." <malsyned <at> malsyned.net>:

> 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.
>

[99-color-escapes-every-line-v2.patch (text/x-patch, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.