On Fri, 18 Dec 2020 14:26:32 GMT, Hannes Wallnöfer <hann...@openjdk.org> wrote:
> Jon, > > I created a new pull request for this on the jdk16 repository because the > original one was created on the mainline jdk repo. The first commit in this > PR is identical to the one you already reviewed in the old PR. The second > commit adds checks and tests for changes from non-final to final and changes > in abstractness of overriding methods. Thus the full list of conditions under > which a simple overriding method is included in the details section of the > overriding class is now: > > > - method access changes from protected to public > - method is overridden as final > - method overridden with different abstractness > - any change in thrown exceptions > - change in documented annotations anywhere in the method signature This pull request has now been integrated. Changeset: 45bd3b9e Author: Hannes Wallnöfer <hann...@openjdk.org> URL: https://git.openjdk.java.net/jdk16/commit/45bd3b9e Stats: 472 lines in 7 files changed: 443 ins; 15 del; 14 mod 8223607: --override-methods=summary ignores some signature changes Reviewed-by: jjg ------------- PR: https://git.openjdk.java.net/jdk16/pull/52
