Closed
Bug 329924
Opened 19 years ago
Closed 19 years ago
ignore-whitespace option in interdiff page
Categories
(Bugzilla :: Attachments & Requests, enhancement)
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.
Comment 1•19 years ago
|
||
*** This bug has been marked as a duplicate of 329923 ***
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → DUPLICATE
![]() |
||
Updated•19 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 2•19 years ago
|
||
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 → ---
![]() |
||
Comment 3•19 years ago
|
||
(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 ago → 19 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•