RFR: 8338851: Hoist os::Posix::realpath() to os::realpath() and implement on Windows [v20]
Simon Tooke
stooke at openjdk.org
Wed Oct 16 12:38:15 UTC 2024
On Tue, 15 Oct 2024 23:52:35 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Simon Tooke has updated the pull request incrementally with one additional commit since the last revision:
>>
>> clean up test code
>
> test/hotspot/gtest/runtime/test_os.cpp line 422:
>
>> 420: errno = 0;
>> 421: const char* returnedBuffer = os::realpath(nosuchpath, buffer, sizeof(nosuchpath) - 2);
>> 422: /* Returns ENOENT on Linux, ENAMETOOLONG on Windows */
>
> Suggestion:
>
> /* Reports ENOENT on Linux, ENAMETOOLONG on Windows */
>
> and similarly below.
The difference is important, thanks - fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20683#discussion_r1803022082
More information about the hotspot-dev
mailing list