Enhanced diff viewer

[ludde]

Add an option to display .diff-files in the same way that diffs in a changeset are shown.

[xris@…]

I second this idea. trac's changeset viewer is much easier to read than a diff file.

[ludde]

Functionality provided by this patch:

​http://projects.edgewall.com/trac/attachment/wiki/LudvigStrigeus/colordiff.2.py

[Christian Boos]

I tested the patch a little bit, it works well for .diff files, but it exits too early when processing inline diff blocks in the Wiki: it exits after scanning a few lines, e.g. if I output the lines that are scanned, I get:

1 Index: trac/versioncontrol/svn_fs.py
2 ===================================================================
3 --- trac/versioncontrol/svn_fs.py       (revision 1535)
4 +++ trac/versioncontrol/svn_fs.py       (working copy)
5 @@ -55,49 +55,44 @@
6

6 is an empty line, and then it stops…

[Christopher Lenz]

Line 6 should containing a leading space, i.e. it should not be empty for a proper diff.

[Christopher Lenz]

Implemented in [1622]. Thanks for the patch!

[Christian Boos]

I nevertheless think it's worth supporting the patches containing empty lines, since it's pretty common to insert small diffs in wiki pages that you just generated with a command line utility (e.g. copy/pasting the output of svn diff).

In a Windows console (cmd.exe or even Putty), that leading space is lost.

I checked xterm on Linux and it works properly (i.e. the leading space is preserved), but Konsole (1.2.3, KDE 3.1.4) also fails to preserve that leading space.