Hi Robert,

We purposefully do not provide this feature for diff uploads. From the Review Board diff viewer, you can hide lines that are whitespace-only changes (okay, this is broken in the past couple releases, but the next one will fix it).

We prefer to do customization of display like that on the Review Board side, and in fact, it's very important that Review Board does not have a mangled view of how the diff looks, since we apply the patch and do some processing and caching on the result.


Robert Dailey wrote:
Is there a way to tell post-review to ignore whitespace in diffs that
it uploads? I use this tool through P4V (perforce visual client).
Sometimes a file is full of tabs to spaces conversions and it mixes in
with real, useful code changes.

I'd like to see a way to upload a diff without whitespace OR a
user-option in ReviewBoard to hide diffs on lines with only whitespace

Anyone? Thanks.

Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
Beanbag, Inc. - http://www.beanbaginc.com

Want to help the Review Board project? Donate today at 
Happy user? Let us know at http://www.reviewboard.org/users/
To unsubscribe from this group, send email to 
For more options, visit this group at 
--- You received this message because you are subscribed to the Google Groups "reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to