Re: cvs: php-src /ext/standard dl.c

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? Something like:

     zend_error(E_STRICT, "dl() is deprecated - use 'extension=%s' in your php.ini", Z_STRVAL_PP(file));

Would print out: Strict Standards: dl() is deprecated - use 'extension=/path/to/foo.so' in your php.ini in /path/to/file.php on line xy Note: I also added '' but that is just a matter of taste I guess. Users might copy the single quotes into the php.ini and complain about errors.

Yeah I think that would be nice (it might not always be 100% accurate because dl() uses CWD and I think php.ini needs full-path (or Apache CWD) but it would probably still make it look nicer. Andi