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