On Wed, 16 Oct 2024 16:56:38 GMT, Brian Burkhalter <[email protected]> wrote:
> Modify the `handleAvailable` native code path to ignore an
> `ERROR_INVALID_FUNCTION` encountered by `PeekNamedPipe` and update an
> existing test to verify this change.
src/java.base/windows/native/libjava/io_util_md.c line 357:
> 355: if (lastError != ERROR_BROKEN_PIPE &&
> 356: lastError != ERROR_INVALID_FUNCTION) {
> 357: return FALSE;
What would you think about not special case these errors and instead pbytes=0
and TRUE if PeekNamedPipe fails.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21543#discussion_r1866472851