On Mon, Apr 9, 2018 at 12:02 PM Dennis Lambe Jr. <malsy...@malsyned.net> wrote: > > 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." <malsy...@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.
Thank you for the contribution. I've written a ChangeLog entry for you and attached a rebased commit. However, the change is large enough that we'll also need a copyright assignment from you. Can you do that? Here are instructions: https://git.savannah.gnu.org/cgit/gnulib.git/tree/doc/Copyright/request-assign.changes
ansi-reset-less.diff
Description: Binary data
