bug-diffutils
[Top][All Lists]
Advanced

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

[bug-diffutils] bug#31105: bug#31105: bug#31105: [PATCH] Restart and res


From: Jim Meyering
Subject: [bug-diffutils] bug#31105: bug#31105: bug#31105: [PATCH] Restart and reset ANSI escapes on each line for compatibility with less -R
Date: Sun, 30 Dec 2018 16:15:28 -0800

On Sat, Dec 29, 2018 at 2:45 PM Dennis Lambe Jr. <address@hidden> wrote:
...
> Sure. I don't know if this qualifies as a bug fix, or new feature, or
> what. But how about this?
>
>    diff --color now produces output compatible with less -R.

Thanks. I put that in a NEWS section labeled "Improvements."
I've made some trivial SP/TAB indentation changes in an attempt to
reach consistency (the existing code is not consistent on that front),
removed the braces you added around a single-statement "then" block,
and made this change:
-  fprintf (outfile, "\n");
+  putc ('\n', outfile);

Re TAB-vs-SP indentation, after the upcoming release, I will make the
global substitution to convert all indentation TABs to spaces.

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.

Attachment: diff-less-color.diff
Description: Binary data


reply via email to

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