Closed (fixed)
Project:
Autoload
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Jan 2009 at 06:43 UTC
Updated:
31 Mar 2009 at 22:00 UTC
Great module! I began writing a similar one before I found this one.
One thought:
Using require_once() in the autoload_class() function is redundant. autoload_class() is only called when php can't find a class definition. If the file containing a class was already included, the class defenition would already be loaded and autoload_class() would not be called. Also, many articles on php performance and optimization recommend not using require_once and include_once wherever possible because they are much more expensive operations.
Comments
Comment #1
Crell commentedThat's probably true, yes. Can you roll a quick patch to change that, and test it to make sure there's no mystery bugs we aren't thinking of?
Comment #2
Crell commentedCommitted.