Re: svn commit: r1894917 - /apr/apr/trunk/poll/unix/wakeup.c

2021-11-16 Thread Ruediger Pluem
On 11/17/21 2:39 AM, Mladen Turk wrote: > > > On 16/11/2021 12:00, Yann Ylavic wrote: >> On Wed, Nov 10, 2021 at 4:19 PM Yann Ylavic wrote: >>> >>> Otherwise I'll revert because I have no way to test it, but I think >>> that apr_poll_drain_wakeup_pipe() might block on Windows for the same

Re: svn commit: r1894917 - /apr/apr/trunk/poll/unix/wakeup.c

2021-11-16 Thread Mladen Turk
On 16/11/2021 12:00, Yann Ylavic wrote: On Wed, Nov 10, 2021 at 4:19 PM Yann Ylavic wrote: Otherwise I'll revert because I have no way to test it, but I think that apr_poll_drain_wakeup_pipe() might block on Windows for the same reason as r1894914 for platforms with poll()able pipes.

Re: svn commit: r1894917 - /apr/apr/trunk/poll/unix/wakeup.c

2021-11-16 Thread Mladen Turk
On 16/11/2021 12:00, Yann Ylavic wrote: On Wed, Nov 10, 2021 at 4:19 PM Yann Ylavic wrote: Otherwise I'll revert because I have no way to test it, but I think that apr_poll_drain_wakeup_pipe() might block on Windows for the same reason as r1894914 for platforms with poll()able pipes.

Re: svn commit: r1894917 - /apr/apr/trunk/poll/unix/wakeup.c

2021-11-16 Thread Yann Ylavic
On Wed, Nov 10, 2021 at 4:19 PM Yann Ylavic wrote: > > Otherwise I'll revert because I have no way to test it, but I think > that apr_poll_drain_wakeup_pipe() might block on Windows for the same > reason as r1894914 for platforms with poll()able pipes. Reverted in 1.7.x, I'll leave it in trunk