logo       

Re: [PECL-CVS] Re: cvs: pecl /iptcdata iptcdata.c: msg#00049

php.pecl.devel

Subject: Re: [PECL-CVS] Re: cvs: pecl /iptcdata iptcdata.c

ok thanks Derick and Mike. I will change it to E_RECOVERABLE_ERROR then ok?
I am still learning!

2006/11/17, Derick Rethans <derick@xxxxxxx>:
On Fri, 17 Nov 2006, Michael Wallner wrote:

> Manfred Weber wrote:
> > crafics Fri Nov 17 13:35:10 2006 UTC
> >
> > Modified files:
> > /pecl/iptcdata iptcdata.c
> > Log:
> > added proper error handler
>
> Are you sure about all those fatal E_ERRORs? *shudders*

They should atleast be E_RECOVERABLE_ERRORs, otherwise E_WARNINGs...
E_ERROR should only be used if the engine ends up in an unstable status
(which is clearly not the case here).

regards,
Derick
--
Derick Rethans
http://derickrethans.nl | http://ez.no | http://xdebug.org




<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise