RFR: 7903872: Option testThreadFactoryPath support relativepath [v2]
SendaoYan
syan at openjdk.org
Mon Oct 21 12:22:11 UTC 2024
On Mon, 21 Oct 2024 10:30:21 GMT, Christian Stein <cstein at openjdk.org> wrote:
>> src/share/classes/com/sun/javatest/regtest/config/RegressionParameters.java line 1291:
>>
>>> 1289:
>>> 1290: public void setTestThreadFactoryPath(String testThreadFactoryPath) {
>>> 1291: this.testThreadFactoryPath = new File(testThreadFactoryPath).getAbsolutePath();
>>
>> Let's use NIO `Path` here.
>>
>> Suggestion:
>>
>> this.testThreadFactoryPath = Path.of(testThreadFactoryPath).toAbsolutePath().toString();
>
> Background: [7903091: Convert jtreg to use NIO](https://bugs.openjdk.org/browse/CODETOOLS-7903091)
Okay, thanks for the remind. The java.io.File has been replaced as NIO.
-------------
PR Review Comment: https://git.openjdk.org/jtreg/pull/233#discussion_r1808685339
More information about the jtreg-dev
mailing list