Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2008-05-16 Thread Felipe Pena
Em Sex, 2008-05-16 às 11:40 +0400, Antony Dovgal escreveu: > On 16.05.2008 07:11, Felipe Pena wrote: > > - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &filename) == > > FAILURE) { > > + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z/", &filename) > > == FAILURE) { > >

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2008-05-16 Thread Antony Dovgal
On 16.05.2008 07:11, Felipe Pena wrote: - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &filename) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z/", &filename) == FAILURE) { return; } + + convert_to_string(fil

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2007-09-13 Thread Stanislav Malyshev
Wouldn't something with strcspn and a string build form '/' and DEFAULT_SLASH be faster? Not sure. I would rather use strpbrk, though I'm not sure how easy is to actually build that string from macros. But dl() is anyway not the shining star of performance, so saving a dozen of CPU cycles whi

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2007-09-13 Thread Marcus Boerger
Hello Stanislav, Thursday, September 13, 2007, 3:16:24 AM, you wrote: > stasThu Sep 13 01:16:24 2007 UTC > Modified files: > /php-src/ext/standard dl.c > Log: > MFB: docs say dl() parameter can be filename only - enforce it > > > http://cvs.php.ne

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2005-06-16 Thread Dan Scott
On 6/16/05, Wez Furlong <[EMAIL PROTECTED]> wrote: > The PDO tests; Cc'd to the PDO test gurus. > > --Wez. > > Derick Rethans wrote: > > On Thu, 16 Jun 2005, Wez Furlong wrote: > > > > > >>wez Thu Jun 16 11:36:40 2005 EDT > >> > >> Modified files: > >>/php-src/ext/standard dl.c

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2005-06-16 Thread Wez Furlong
The PDO tests; Cc'd to the PDO test gurus. --Wez. Derick Rethans wrote: On Thu, 16 Jun 2005, Wez Furlong wrote: wez Thu Jun 16 11:36:40 2005 EDT Modified files: /php-src/ext/standard dl.c Log: Don't emit the E_STRICT dl-is-deprecated notice on SAPI's where

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2005-06-16 Thread Derick Rethans
On Thu, 16 Jun 2005, Wez Furlong wrote: > wez Thu Jun 16 11:36:40 2005 EDT > > Modified files: > /php-src/ext/standard dl.c > Log: > Don't emit the E_STRICT dl-is-deprecated notice on SAPI's where dl() is > widely (and safely) used. > This allows our test

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2004-02-14 Thread Pierre-Alain Joye
On Sat, 14 Feb 2004 20:17:22 +0100 Jan Lehnardt <[EMAIL PROTECTED]> wrote: > Hi Pierre > On 14 Feb 2004, at 20:03, Pierre-Alain Joye wrote: > > The best way to improve it is to remove it ;) > > ranting helps. Please see internals, this is unrelated to > the discussion. I cannot resist :) -- PH

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2004-02-14 Thread Jan Lehnardt
Hi Pierre On 14 Feb 2004, at 20:03, Pierre-Alain Joye wrote: The best way to improve it is to remove it ;) ranting helps. Please see internals, this is unrelated to the discussion. Jan -- -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2004-02-14 Thread Pierre-Alain Joye
On Sat, 14 Feb 2004 19:02:23 - "Jan Lehnardt" <[EMAIL PROTECTED]> wrote: > Modified files: > /php-src/ext/standard dl.c > Log: >- improve error message The best way to improve it is to remove it ;) pierre -- PHP CVS Mailing List (http://www.php.net/) To unsub

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2004-02-14 Thread Andi Gutmans
At 06:54 PM 2/14/2004 +0100, Jan Lehnardt wrote: Hi, On 11 Feb 2004, at 12:46, Zeev Suraski wrote: + zend_error(E_STRICT, "dl() is deprecated - use extension= in your php.ini"); unrelated to the ongoing discussion, wouldn't it make sense to show the module's name in the error message

Re: [PHP-CVS] cvs: php-src /ext/standard dl.c

2004-02-14 Thread Jan Lehnardt
Hi, On 11 Feb 2004, at 12:46, Zeev Suraski wrote: + zend_error(E_STRICT, "dl() is deprecated - use extension= in your php.ini"); unrelated to the ongoing discussion, wouldn't it make sense to show the module's name in the error message? Something like: zend_error(E_STRICT, "dl() is deprecate