[dev] [commits] Horde branch master updated. 67188414002f31bb1a9635e2abdae8d1c0312488

[Gunnar Wrobel]

Hi Chuck,

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Gunnar Wrobel <p at rdus.de>:
>
>> commit 67188414002f31bb1a9635e2abdae8d1c0312488
>> Author: Gunnar Wrobel <p at rdus.de>
>> Date:   Mon Aug 23 17:26:52 2010 +0200
>>
>>    I don't see a reason to avoid providing the default autoloader within
>>    Horde_Autoloader.
>>
>>    It introduces no additional dependencies and will be required by a any
>>    application that does not derive from Horde_Core.
>
> Those apps could provide their own autoloader, though.

They still can. I just didn't see the point in duplicating the code  
for those ones that will use the default variant. And depending on  
Horde_Core is something I would like to avoid for anything that is not  
a full blown horde app.

>
> I don't feel super strongly about this, but if it sticks, you should  
> remove the one in Core also, since it's an exact duplicate.

Did that now. Thanks for the feedback!

Cheers,

Gunnar

>
> -chuck
>
> -- 
> Horde developers mailing list - Join the hunt: http://horde.org/bounties/
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe at lists.horde.org
>