On Tue, 26 Nov 2024 14:57:18 GMT, Nizar Benalla <nbena...@openjdk.org> wrote:

>> Please review this patch to improve the javadoc preview header. The parent 
>> bug include more details.
>> 
>> This is meant to prevent a bug where, if a non preview class extend an 
>> `@PreviewFeature` interface, the Javadoc displays `Programs can only use 
>> <<Type>> when preview features are enabled` when the API can be used without 
>> enable preview features as long as we don't refer to any methods in the 
>> supertype
>> 
>> This is specifically the case in  #17543. Here is the javadoc output 
>> [AsymmetricKey](https://cr.openjdk.org/~nbenalla/GeneratedDocs/8343239-2/docs/api/java.base/java/security/AsymmetricKey.html)
>>  after merging this branch with the PR `298420: PEM API: Implementation 
>> (Preview) #17543`. The only files affected by this change are the class 
>> `DEREncodable` and it's Subinterfaces.
>> 
>> One other bug I noticed is that, on the `DEREncodable` page, it's direct 
>> subinterfaces had the superscript `PREVIEW` link. I've added a condition to 
>> remove it.
>
> Nizar Benalla has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Changed based on Hannes' changes

I had to resolve a merge conflict and update a test after 
[JDK-8318416](https://bugs.openjdk.org/browse/JDK-8318416), I've also made 
changes based on chen's suggestions.

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

PR Comment: https://git.openjdk.org/jdk/pull/22126#issuecomment-2503407501

Reply via email to