Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend.c zend_builtin_functions.c zend_execute_API.c zend_globals.h php4/tests/lang 035.phpt
| From: | Andi Gutmans | Date: | Fri, 16 Aug 2002 07:10:12 +0000 |
| Subject: | Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend.c zend_builtin_functions.c zend_execute_API.c zend_globals.h php4/tests/lang 035.phpt | ||
| References: | 1 | Groups: | php.zend-engine.cvs |
| Request: | Send a blank email to php-zend-engine-cvs+get-317@lists.php.net to get a copy of this message | ||
Hey,
I remember it was decided on the engine 2 list not to register an exception class ourselves but to allow every PHP project to decide on its own how to register it. This is as every project might want a different interface for its base exception class.
Also, I think when your exception handler is called, if it throws an exception itself, you're not freeing the first exception. You should probably free params[0] there.
Last but not least, please always use tabs. A few lines there are using spaces.
Thanks,
Andi
At 12:41 AM 8/16/2002 +0000, Stig Bakken wrote:
ssb Thu Aug 15 20:41:36 2002 EDTModified files:/php4/tests/lang 035.phpt /ZendEngine2 zend.c zend_builtin_functions.c zend_execute_API.c zend_globals.hLog: @- Added set_exception_handler() function for registering a global, @ catch-all exception handling function (Stig) - Added set_exception_handler() function for registering a global,catch-all exception handling function (Stig)-- Zend Engine CVS Mailing List (http://cvs.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php