Closed Bug 1323011 Opened 8 years ago Closed 6 years ago

Diff viewer thinks chunks got moved in a diff that just removes a bunch of code

Categories

(MozReview Graveyard :: Review Board: DiffViewer, defect)

Product:
MozReview Graveyard
Component:
Review Board: DiffViewer
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: bzbarsky, Unassigned)

Details

Look at the dom/promise/Promise.cpp diff in https://reviewboard.mozilla.org/r/98088/diff/1#index_header and search for "moved from".  Observe the complexity of the next several dozen lines.  Compare to the raw diff: all that's going on here is code removal.
MozReview is now obsolete. Please use Phabricator instead. Closing this bug.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.