On Thu, 4 Apr 2024 21:21:58 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

>> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/ConstructorWriter.java
>>  line 200:
>> 
>>> 198:         content.add(heading);
>>> 199:         return HtmlTree.SECTION(HtmlStyle.detail, content)
>>> 200:                 .setId(memberAnchor);
>> 
>> It's a bit disappointing that more of this isn't in `HtmlIds`.
>> It feels like it perpetuates the original ugly code.
>> 
>> Would it make sense for `htmlIds` to return a record/pair containing both 
>> the `memberId` and `erasureId` for `ExecutableElement` ?
>
> I'll see how it pans out.

Please have a look at the recent update to see if it's to your liking. Thanks.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18519#discussion_r1553818424

Reply via email to