Re: Optional nowait argument to sem_acquire

From: 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

« previous php.internals (#77037) next »