Have binary files in the changelist. Create a diff using p5 diff -du & upload the diff manually to create the review. The reviewboard displays: "This is a binary file. The content cannot be displayed." without the type of perforce change.
This is more of an enhancement than a bug, I'd argue. There's not always much info we can get for binary files. I'll look into adding this.
- Type-Defect + Type-Enhancement + Component-DiffViewer
I'd also like to see this. While the general case may be difficult to display properly (though one could imagine something involving xxd followed by diff, or allowing special filters based on filetype the way that CUPS might interpret PDF files differently than PS files), it should be possible to handle additions and deletions more intelligently. For example, a p4 deletion can easily be displayed as "file deleted" in the web UI and encoded in the diff correctly, regardless of file type.
We're using 1.0.8 with a subversion database, and a post-commit workflow. Uploading a diff with an image delete, image add, and image modify (3 different files) only shows "binary file" for each one, so the reviewer can't tell which of the three files is the one being deleted. If you could show the before and after images, great, but if not, showing if it's an add/delete/modify would help alot.