You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Diff parsing misinterprets file names for standard diff command #557
The parser swallows the timestamps as part of the filename, too, which (1) is distracting clutter and (2) breaks syntax highlighting since it can't interpret the file type from the suffix anymore.
getFilename in diff-parser.js should be able to strip off these suffixes. I think they're always rendered with a \t character between the filename and the time, so it should be pretty straightforward.
I was actually reading the generated installed JS and didn't notice the comment in the TS source that the regex at the end is already intended to strip off these dates.
So update on the above issue: I'm not immediately sure why, but this doesn't work at least with /usr/bin/diff on macOS, which outputs a header exactly as shown in the issue.
Activity
jrk commentedon Apr 7, 2025
I was actually reading the generated installed JS and didn't notice the comment in the TS source that the regex at the end is already intended to strip off these dates.
So update on the above issue: I'm not immediately sure why, but this doesn't work at least with
/usr/bin/diffon macOS, which outputs a header exactly as shown in the issue.