Description: ------------ Wherever we have: HTML_QuickForm2_Loader::loadClass(...); change to: if (!class_exists($class)) { HTML_QuickForm2_Loader::loadClass(...); } By not specifying the second parameter (i.e., false), any registered autoloaders (including HTML_QuickForm_Loader::autoload) will have first crack at loading the class.

I don't see any benefit here as a) loadClass() immediately calls class_exists() itself so the only overhead is the method call; b) the provided autoload implementation uses the same loadClass() Please bear in mind that PEAR1 isn't exactly designed for autoload.

I hope you'll reconsider. The difference is the absence of the second parameter (false). Our app implements its own autoloader that follows the PEAR naming convention, but doesn't rely on the include_path. By calling class_exists() without the second parameter (false), our autoloader is called. p.s. this idiom is seen throughout ZendFramework, so this isn't a novel idea, i.e., if (!class_exists($className)) { Zend_Loader::loadClass($className); }