search

rlwhitcomb / utilities

Some of my personal utility programs
MIT License
2 stars 0 forks source link

Tester should implement line-mode diffs (either by default or as an option) #138

Open rlwhitcomb opened 2 years ago

rlwhitcomb commented 2 years ago

Currently Tester displays character-mode diffs between the canon file(s) and the actual output/error. This format can be hard to read and interpret, and I often find myself using the reported temp file names along with "diff" or "fc" (platform-dependent) to get an easier to read line-mode diff. The diff-match-patch library we're using for the diffs can do line-mode diffs (see here: https://github.com/google/diff-match-patch/wiki/Line-or-Word-Diffs), so at least Tester should have an option for this, if not default or only do the diffs this way. Maybe default should be line-mode with an option for character-mode...

rlwhitcomb commented 2 years ago

It appears the relevant methods and structure are all marked "protected" in the source for diff-match-patch (at least the code we have copied), so we will have to make public in order to use it. Should check for updates of this source...