On Sat, Jun 3, 2017 at 7:15 PM, William A Rowe Jr <[email protected]> wrote:
> On Jun 3, 2017 10:53, "Yann Ylavic" <[email protected]> wrote:
>
>
> Right, but why apr_os_mutex_set[_ex](DEFAULT_TIMED) should fail on
> Windows, OS2 and BEOS?
> APR_LOCK_DEFAULT_TIMED = APR_LOCK_DEFAULT allows this, and does not
> break with _get[_ex]() either I think.
>
>
> How do you mean, it is allowed right now with the current implementations?
APR_DECLARE(apr_status_t) apr_os_proc_mutex_put_ex(apr_proc_mutex_t **pmutex,
apr_os_proc_mutex_t *ospmutex,
apr_lockmech_e mech,
int register_cleanup,
apr_pool_t *pool);
It takes a mech as argument, the previous implementation allowed to
specify DEFAULT_TIMED not only for unixes, but also for Windows, OS2
and BEOS.
But now it would fail, which does not really help user code portability IMHO.