MozReview doesn't show any indication for end-of-line character changes -- doesn't even display the changed lines

NEW
Unassigned

Status

MozReview
Review Board: DiffViewer
a year ago
a year ago

People

(Reporter: dholbert, Unassigned)

Tracking

Details

(Reporter)

Description

a year ago
STR:
 1. Use MozReview's diff-viewer to view a patch that modifies line endings (e.g. changing from DOS-style to UNIX-style).  For example, here's a *2MB* diff which changes the line-ending for nearly every line in the affected file:
 https://reviewboard.mozilla.org/r/116740/diff/1

ACTUAL RESULTS:
MozReview's diff viewer shows only the last 8 lines of each file, and it doesn't show any changes, and it simply says "This file contains only whitespace changes."

EXPECTED RESULTS:
MozReview's diff viewer should show *all* of the changed lines, and it should highlight the fact that they're being changed somehow.

For reference, here's the raw-patch version of the diff linked above:
 https://reviewboard.mozilla.org/r/116740/diff/1/raw/
It shows "-"/ "+" lines for the (many) whitespace changes here.
(Reporter)

Comment 1

a year ago
As a motivator, this caused real confusion over on bug 1339298, where a rebase operation accidentally restored a bunch of DOS endline characters which had been removed on mozilla-central.

The extremely-deceptive diff is shown here -- look at the ContentPrefs.cpp change in particular:
 https://reviewboard.mozilla.org/r/112904/diff/3/

That looks like just a trivial 1-line insertion -- but really, the commit also touches **every line in that file**, as you can see from the raw diff:
 https://reviewboard.mozilla.org/r/112904/diff/3/raw/
You need to log in before you can comment on or make changes to this bug.