Closed Bug 329923 Opened 18 years ago Closed 18 years ago

ignore-whitespace option for patch diff viewer

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 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.
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
*** 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.