Re: Optional nowait argument to sem_acquire
| From: | guilhermeblanco@gmail.com | Date: | Tue, 02 Sep 2014 13:38:37 +0000 |
| Subject: | Re: Optional nowait argument to sem_acquire | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-77037@lists.php.net to get a copy of this message | ||
Hi,
Since I don't know where you guys like to discuss implementations, I'm
pasting same thing I did in php-src PR:
I'd propose instead of accepting "nowait" only, I'd use a better support
and accept "until", dealing with possible cases:
- -1: Blocks and wait until lock is acquired
- 0: Exactly what your patch does, attempt to grab and if it fails, just
return false (I'd remove the E_WARNING)
- > 0: Acceptable time in milliseconds it would wait until consider the
acquire failed
Cheers,
On Tue, Sep 2, 2014 at 8:50 AM, Matteo Beccati <php@beccati.com> wrote:
> Hi everyone,
>
> the following patch:
>
> https://github.com/php/php-src/pull/741
>
> has been lying around somewhere on my box since years and I recently
> stumbled upon it.
>
> However 5.6 was already RC and Ferenc suggested to eventually wait for
> 5.6.1.
>
> Do you think it's ok to add such a minor feature to 5.6.1?
>
>
> Cheers
> --
> Matteo Beccati
>
> Development & Consulting - http://www.beccati.com/
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>
--
Guilherme Blanco
MSN: guilhermeblanco@hotmail.com
GTalk: guilhermeblanco
Toronto - ON/Canada