> 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. > >  > > After the patch, `testA()` is only documented once: > > 
Nizar Benalla has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 17 additional commits since the last revision: - review feedback: add positive test - Merge branch 'master' into duplicate-APIs - Merge branch 'master' into duplicate-APIs - Feedback from @hns - Expand test with new test cases - Merge branch 'master' into duplicate-APIs - revert back to simple earlier fix - revert back to master - Merge branch 'master' into duplicate-APIs - Merge branch 'master' into duplicate-APIs - rename test to be more accurate and simplify code - ... and 7 more: https://git.openjdk.org/jdk/compare/f8ee11ec...00e7adcc ------------- Changes: - all: https://git.openjdk.org/jdk/pull/25123/files - new: https://git.openjdk.org/jdk/pull/25123/files/caeb20ad..00e7adcc Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=25123&range=07 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25123&range=06-07 Stats: 51882 lines in 1215 files changed: 35082 ins; 10971 del; 5829 mod Patch: https://git.openjdk.org/jdk/pull/25123.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/25123/head:pull/25123 PR: https://git.openjdk.org/jdk/pull/25123