RFR: 8224052: Javadoc doesn't handle non-public intermediate types well

Jonathan Gibbons jonathan.gibbons at oracle.com
Thu Jan 16 18:37:28 UTC 2020


If you can get this in under the wire for 14, go for it.

Otherwise, go for 15.

-- Jon

On 01/16/2020 05:42 AM, Hannes Wallnöfer wrote:
> Yes, the fact that this is allowed by javac was what lead me to include this in the patch.
>
> Unfortunately the webrev I sent was against JDK 15 instead of 14. I uploaded a new webrev for 14. Merging was straightforward and no changes were required for the code. Only test output had to be adapted to the old HTML anchor style.
>
> http://cr.openjdk.java.net/~hannesw/8224052/webrev.01/
>
> This also adds the copyright header that was missing in the new test file as noted by Sundar.
>
> Hannes
>
>
>> Am 16.01.2020 um 02:14 schrieb Jonathan Gibbons <jonathan.gibbons at oracle.com>:
>>
>>
>>
>> On 01/15/2020 03:47 PM, Jonathan Gibbons wrote:
>>> As regards package-private and private intermediate supertypes, in the absence of any pre-existing comment to the contrary, your inclusion of isPrivate in isUndocumentedEnclosure is morally justifiable, even if it is a change in behavior in an absurdly weird corner case that (to the best of my knowledge) is undocumented.
>> As further justification for this change in behavior to be regarded as a bug fix, I note that javac allows access to public methods of private intermediate supertypes.  This implies the behavior is required by JLS, and so is justification for javadoc documenting the available methods.
>>
>> Toy files attached, for the record.
>>
>> -- Jon
>> <Client.java><Demo.java>



More information about the javadoc-dev mailing list