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

[Jan Schneider]

Zitat von Chuck Hagenbuch <chuck at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>>> 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.
>
> I agree. Also, if the goal is to get rid of __FILE__ and __LINE__ (a  
> good goal), we can just use debug_backtrace() to get that.

What's wrong with __FILE__ and __LINE__? Passing debug_backtrace()  
around seem like overhead to me. Or you just talking about  
Horde::fatal()?

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/20090715/46e5c6d9/attachment-0001.bin>