[cvs] commit: framework/Imap_Client/lib/Horde/Imap/Client Socket.php
Chuck Hagenbuch
chuck at horde.org
Tue Oct 28 14:29:01 UTC 2008
Quoting Jan Schneider <jan at horde.org>:
> Zitat von Chuck Hagenbuch <chuck at horde.org>:
>
>> Quoting Michael M Slusarz <slusarz at horde.org>:
>>
>>> slusarz 2008-10-27 17:58:27 EDT
>>>
>>> Modified files:
>>> Imap_Client/lib/Horde/Imap/Client Socket.php
>>> Log:
>>> Fix require_once lib.
>>
>> This library, and other PHP 5 libs, should be using autoload for this.
>
> Wouldn't it make more sense to use the autoloader only in
> applications, to not add another dependency and to allow faster
> class name lookups/library loading?
But we're not adding a specific library dependency. We're not even
completely relying on autoload; it's possible to use
get_included_files to generate a static include list for production
use if that's an issue. I think it is a bigger win to use autoload in
the framework than in applications, if we absolutely have to choose
one. This makes the framework libraries independent from any specific
code loading strategy. Autoload is great for most instances, and files
can be included with full paths by the code _using_ the package if
serious optimization is an issue.
-chuck
More information about the cvs
mailing list