On Tue, 10 Mar 2026 10:16:58 GMT, Volkan Yazici <[email protected]> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> use test.main.class
>
> test/jdk/com/sun/net/httpserver/TaskRejectedTest.java line 48:
>
>> 46: * @bug 8169358
>> 47: * @summary HttpServer does not close client connection when
>> RejectedExecutionException occurs.
>> 48: * @run main/othervm TaskRejectedTest
>
> Can we use `${test.main.class}` here and all other `@run` tags, please?
I had considered that, but was hesitant to do it. The use of
`${test.main.class}` I think is understandable for a new test class, but I
think it isn't too useful to existing test classes which have the right name
already and which aren't being renamed.
I've however updated this PR to apply your suggestion.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30158#discussion_r2912336602
