RFR: 8338851: Hoist os::Posix::realpath() to os::realpath() and implement on Windows [v11]
Simon Tooke
stooke at openjdk.org
Thu Sep 19 07:29:38 UTC 2024
On Thu, 19 Sep 2024 04:27:54 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Simon Tooke has updated the pull request incrementally with one additional commit since the last revision:
>>
>> remove tabs
>
> test/hotspot/gtest/runtime/test_os.cpp line 453:
>
>> 451: errno = 0;
>> 452: returnedBuffer = os::realpath(tmppath, buffer, sizeof(buffer));
>> 453: EXPECT_TRUE(errno == EINVAL);
>
> How is this an EINVAL case?
You are correct - this was a cut and paste error; the call tested should have been
`os::realpath(tmppath, nullptr, sizeof(buffer));`
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20683#discussion_r1766304627
More information about the serviceability-dev
mailing list