RFR: 8344942: Template-Based Testing Framework [v2]
Emanuel Peter
epeter at openjdk.org
Mon Mar 31 11:23:43 UTC 2025
On Mon, 31 Mar 2025 11:04:07 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> test/hotspot/jtreg/compiler/lib/template_framework/Template.java line 41:
>>
>>> 39: * the {@link Template}s provide <strong>hashtag replacements</strong> in the Strings: the {@link Template} argument
>>> 40: * names are captured, and the argument values automatically replace any {@code "#name"} in the Strings. See the
>>> 41: * different overloads of {@link make} for examples. Additional hashtag replacements can be defined with {@link let}.
>>
>> General Javadocs comment: You should preceed method names with `#` in order to create a proper link.
>>
>> Without:
>> 
>>
>> With (i.e. `{@link make}` and `{@link #let})`:
>> 
>
> interesting. As we saw offline, `javadoc` does not need the hashtag, but some editors do, and it seems to be common practice to have the hashtag. I'll add them for every applicable link.
I think I fixed all, but I cannot easily confirm. Can you check if there are any left?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24217#discussion_r2020862929
More information about the hotspot-compiler-dev
mailing list