Closed
Bug 329923
Opened 18 years ago
Closed 18 years ago
ignore-whitespace option for patch diff viewer
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 a patch's diffs ignoring whitespace changes. For example, look at this set of patch diffs: https://bugzilla.mozilla.org/attachment.cgi?id=213909&action=diff#mozilla/security/nss/tests/ssl/ssl.sh_sec16 This patch changed the indentation level of a lot of code, and changed the actual non-whitespace content of just a few lines. 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 URL that is similar to https://bugzilla.mozilla.org/attachment.cgi?id=213909&action=diff but with some added feature that says "ignore whitespace". Perhaps something likek this: https://bugzilla.mozilla.org/attachment.cgi?id=213909&action=diff&whitespace_mode=ignore 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?id=213909&action=diff too.
Comment 1•18 years ago
|
||
last year the reporter commented to bug 285814 though... :-( *** This bug has been marked as a duplicate of 285814 ***
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → DUPLICATE
Comment 2•18 years ago
|
||
*** Bug 329924 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•