On Fri, 5 Aug 2022 13:36:47 GMT, thyecust <[email protected]> wrote: >> The condition at line 57 (after while) will evaluate to false >> if startLine == null, so the previous if-condition is covered. > > thyecust has updated the pull request with a new target base due to a merge > or a rebase. The incremental webrev excludes the unrelated changes brought in > by the merge/rebase. The pull request contains five additional commits since > the last revision: > > - Merge branch 'openjdk:master' into if > - update the Copyright year > - simplify loop contidition > - Merge branch 'openjdk:master' into if > - remove redundant if-condition after readLine() > > The condition at line 57 (after while) will evaluate to false > if startLine == null, so the previous if-condition is covered.
Thanks for trying :-). I have launched a tier2 job for you. I'll approve the PR if results are successful. ------------- PR: https://git.openjdk.org/jdk/pull/9571
