[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