[dev] [commits] Horde branch master updated. d3a3a8ba4f403b2c05c0573ca519b7e349a8c657

Jan Schneider jan at horde.org
Tue Jul 14 19:32:30 UTC 2009


Zitat von Michael M Slusarz <slusarz at horde.org>:

> -----------------------------------------------------------------------
>
> commit 5ff300faf90378ba5fa0d61a454893813f28e584
> Author: Michael M Slusarz <slusarz at curecanti.org>
> Date:   Tue Jul 14 00:45:30 2009 -0600
>
>     documentation; cleaner fatal() handling
>
>  framework/Core/lib/Horde/Registry.php |   16 ++++++++++------
>  1 files changed, 10 insertions(+), 6 deletions(-)
>
> http://git.horde.org/diff.php/framework/Core/lib/Horde/Registry.php?rt=horde-git&r1=b8577e2c3f7bcf9be4a61ef9bca861f4b04a1083&r2=5ff300faf90378ba5fa0d61a454893813f28e584
>
> -----------------------------------------------------------------------

I don't think it's good practice to use exceptions for anything but  
throwing. Beside that, wrapping in a class when we need the different  
parts (message, file, line) later anyway, is unnecessary IMO.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digitale PGP-Unterschrift
URL: <http://lists.horde.org/archives/dev/attachments/20090714/95544041/attachment.bin>


More information about the dev mailing list