On Tue, 8 Oct 2024 20:46:13 GMT, Chen Liang <li...@openjdk.org> wrote:
>> src/java.base/share/classes/java/lang/constant/ClassDesc.java line 281: >> >>> 279: * >>> 280: * @implSpec >>> 281: * <p>The implementations return the simple name >> >> Suggestion: >> >> * <p>The implementation returns the simple name >> >> >> I think this can be singular as it's not required to have more than 1 >> implementation class. > > Hmm, in fact, should we still have this `@implSpec` if we make this default? I think this can be dropped as `ClassDesc` as well. It can say "This method returns....". See what CSR review would say. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/20665#discussion_r1792509783