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

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php



Reply via email to