RFR: 50: Fix failing tests after e8d32c78dc

Erik Duveblad via github.com duke at openjdk.java.net
Wed Aug 7 10:23:00 UTC 2019


Hi all,

this patch fixes some test failures after e8d32c78dc was integrated. Turns out there are different bugs in git with regards to unified diffs with zero context :cry: @JornVernee fixed the issues with the regular unified diff printer (when there is exactly one parent), this patch works around the issues with git's combined diff printer. The combined diff printer seems to print _correct_ start for empty hunks as long as the file the hunks belong to isn't added.

I also updated a number of tests to take the changes in e8d32c78dc into account.

## Testing
- [x] Run `sh gradlew test` on Linux x86-64

----------------

Commits:
 - 0914742f:	50: Fix failing tests after e8d32c78dc

Pull request:
https://git.openjdk.java.net/skara/pull/39

Webrev:
https://webrevs.openjdk.java.net/skara/39/webrev.00

Patch:
https://git.openjdk.java.net/skara/pull/39.diff

Fetch command:
git fetch https://github.com/openjdk/skara.git pull/39/head:pull/39


More information about the skara-dev mailing list