RFR: 8285368: Overhaul doc-comment inheritance [v2]

Pavel Rappo prappo at openjdk.org
Wed Jun 7 21:04:53 UTC 2023


On Wed, 7 Jun 2023 19:16:51 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

>> Pavel Rappo has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   feedback: make warning less scary
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/Utils.java line 2847:
> 
>> 2845:                     break;
>> 2846:                 }
>> 2847:                 if (isPlainInterface(peek) && !isPublic(peek) && !isLinkable(peek)) {
> 
> What is the significance of the `isPublic` check?
> I'm developing a knee-jerk reaction to such checks, in the face of access-related command-line options.

That directly mimics the current behaviour at addSuperInterfaces(Utils.java:802), which in turn is used at ImplementedMethods(VisibleMemberTable.java:1064) to collect overridden methods in superinterfaces. If you look around, it's quite an idiom and predates this PR.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14357#discussion_r1222173441


More information about the core-libs-dev mailing list