On Fri, 9 May 2025 11:39:44 GMT, Nizar Benalla <nbena...@openjdk.org> wrote:

>> Please review this patch to fix a bug where a method can be documented 
>> multiple times 
>> Consider these 4 classes
>> 
>>                     A       (interface)
>>                    / \
>>                   /   \
>> (abstract class)  C     B   ( interface)
>>                   \   /
>>                    \ /
>>                     D       (class)
>> 
>> 
>> Where `A` declares `testA()`, `C` implements it `public final void testA()`, 
>> `B` extends `A` but does *not* override it, `D` extends `C` and implements 
>> `B`
>> 
>> In the generated javadoc, `testA()` is documented twice.
>> 
>> ![Screenshot 2025-05-08 at 15 51 
>> 19](https://github.com/user-attachments/assets/46538e61-1546-4128-8041-296d8491406f)
>> 
>> After the patch, `testA()` is only documented once:
>> 
>> ![Screenshot 2025-05-08 at 15 52 
>> 16](https://github.com/user-attachments/assets/a6b86a21-7104-460c-8f3f-36cee5962eed)
>
> Nizar Benalla has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   simplify check

I'm not sure about this. @nizarbenalla have you tested whether this changes the 
output for JDK docs?

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

PR Comment: https://git.openjdk.org/jdk/pull/25123#issuecomment-2866404957

Reply via email to