RFR: Handle binary patches in webrev

[Robin Westberg]

On Fri, 20 Sep 2019 09:46:18 GMT, Erik Helin <ehelin at openjdk.org> wrote:

> Hi all,
> 
> this patch makes `git webrev` generate `.patch` files for binary files. Thanks @rwestberg for the initial reproducer!
> 
> ## Testing
> - [x] Added three new unit tests
> 
> Thanks,
> Erik
> 
> ----------------
> 
> Commits:
>  - cd9b801d: Handle binary patches in webrev
> 
>   Stats: 221 lines in 7 files changed: 185 ins; 12 del; 24 mod
> Changes: https://git.openjdk.java.net/skara/pull/150/files
>  Webrev: https://webrevs.openjdk.java.net/skara/150/webrev.00
>   Patch: https://git.openjdk.java.net/skara/pull/150.diff
>   Fetch: git fetch https://git.openjdk.java.net/skara pull/150/head:pull/150

webrev/src/test/java/org/openjdk/skara/webrev/WebrevTests.java line 133:

> 132:             new Webrev.Builder(repo, tmp.path().resolve("webrev")).generate(hash1, hash2);
> 133:         }
> 134:     }

Perhaps add some sort of sanity-checking validation (like the other tests)?

PR: https://git.openjdk.java.net/skara/pull/150