On Wed, 24 Jun 2026 16:29:52 GMT, Chris Plummer <[email protected]> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> remove leftover changes of another bug fix
>
> test/hotspot/jtreg/serviceability/jvmti/vthread/VThreadTest/VThreadTest.java
> line 59:
>
>> 57: } catch (Throwable t) {
>> 58: t.printStackTrace(System.out);
>> 59: throw new RuntimeException("VThreadTest failed: PRODUCER
>> caught a throwable: " + t");
>
> This is failing to compile:
>
> /home/runner/work/jdk/jdk/test/hotspot/jtreg/serviceability/jvmti/vthread/VThreadTest/VThreadTest.java:59:
> error: unclosed string literal
> throw new RuntimeException("VThreadTest failed: PRODUCER caught a
> throwable: " + t");
>
> ^
Thanks! See below.
> test/hotspot/jtreg/serviceability/jvmti/vthread/VThreadTest/VThreadTest.java
> line 70:
>
>> 68: } catch (Throwable t) {
>> 69: t.printStackTrace(System.out);
>> 70: throw new RuntimeException("VThreadTest failed: CONSUMER
>> caught a throwable: " + t");
>
> Same quoting problem on this line.
Good catch, thanks. Fixed now. It was on my plan to run tests before
integration. :)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/31581#discussion_r3469827209
PR Review Comment: https://git.openjdk.org/jdk/pull/31581#discussion_r3469825465
