RFR: webrev: add logging [v2]

Erik Helin ehelin at openjdk.java.net
Thu Jun 18 13:41:43 UTC 2020


> Hi all,
> 
> please review this small patch that add some additional logging to `Webrev` and dereferences `Optional`s a bit more
> carefully in `ModifiedFileView`.
> Testing:
> - [x] `make test` passes on Linux x64
> 
> Thanks,
> Erik

Erik Helin has updated the pull request incrementally with one additional commit since the last revision:

  Use `Collectors.joining`
  
  Co-authored-by: Jorn Vernee <JornVernee at users.noreply.github.com>

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

Changes:
  - all: https://git.openjdk.java.net/skara/pull/674/files
  - new: https://git.openjdk.java.net/skara/pull/674/files/df7bb77c..3dfb8885

Webrevs:
 - full: https://webrevs.openjdk.java.net/skara/674/webrev.01
 - incr: https://webrevs.openjdk.java.net/skara/674/webrev.00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/skara/pull/674.diff
  Fetch: git fetch https://git.openjdk.java.net/skara pull/674/head:pull/674

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


More information about the skara-dev mailing list