On Tue, 20 Jan 2026 17:14:43 GMT, Kevin Walls <[email protected]> wrote:
>> Kieran Farrell has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> relocate print out to SYSTEM section and remove AIX impl
>
> src/hotspot/os/windows/os_windows.cpp line 6262:
>
>> 6260:
>> 6261: void os::print_open_file_descriptors(outputStream* st) {
>> 6262: // File descriptor counting not supported on Windows.
>
> If you said "not implemented" here and in the other comment, it might be
> clearer that somebody could implement these if they build and test it. Not
> supported sounds to me more like a reason we cannot do it.
Windows doesn't support UNIX-style file descriptors hence using the word
'supported', I've updated the AIX stub to say 'not implemented' however.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27971#discussion_r2709837631
