Re: commit: framework/Imap_Client/lib/Horde/Imap/Client Socket.php
Chuck Hagenbuch <[email protected]>
| Newsgroups | gmane.comp.horde.cvs |
|---|---|
| Message-ID | <[email protected]> |
Quoting Jan Schneider <[email protected]>: > Zitat von Chuck Hagenbuch <[email protected]>: > >> Quoting Michael M Slusarz <[email protected]>: >> >>> 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 -- To unsubscribe, mail: [email protected]