Closed Bug 329924 Opened 19 years ago Closed 19 years ago

ignore-whitespace option in interdiff page

Categories

(Bugzilla :: Attachments & Requests, enhancement)

2.19.1
enhancement
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 285814

People

(Reporter: nelson, Unassigned)

Details

As a frequent reviewer of patches, one thing that would help me A LOT would be the ability to view the difference between two patches *ignoring whitespace changes*. For example, look at this interdiff output: https://bugzilla.mozilla.org/attachment.cgi?oldid=213867&action=interdiff&newid=213909&headers=1 The difference between those two patches is primarily a change in indentation of many lines, that is, only whitespace changes. But a few lines had non-whitespace changes as well. I can't see the lines that had non-whitespace changes because they're buried in all those whitespace changes. So, I'd like to have a checkbox that says "ignore whitespace" by the diff button on the page https://bugzilla.mozilla.org/attachment.cgi?id=213909&action=diff The page/script http://bonsai.mozilla.org/cvsview2.cgi has this feature. I'd REALLY like to see it in https://bugzilla.mozilla.org/attachment.cgi?action=interdiff too.
*** This bug has been marked as a duplicate of 329923 ***
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → DUPLICATE
Status: RESOLVED → VERIFIED
Not a duplicate. One bug concerns the "action=diff". The other concerns "action=interdiff". Fixing one will not fix the other.
Status: VERIFIED → REOPENED
Resolution: DUPLICATE → ---
(In reply to comment #2) > The other concerns "action=interdiff". Reread what you wrote in bug 285814 comment 1. You already talked about interdiff in it. We (the Bugzilla team) are not completely blind. We also read comments. ;) *** This bug has been marked as a duplicate of 285814 ***
Status: REOPENED → RESOLVED
Closed: 19 years ago19 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.