[OpenLayers-Trac] [OpenLayers] #2992: move OpenLayers.Util.extend to Class.js

OpenLayers trac-20090302 at openlayers.org
Sat Jan 1 17:57:11 EST 2011


#2992: move OpenLayers.Util.extend to Class.js
---------------------+------------------------------------------------------
 Reporter:  tschaub  |       Owner:              
     Type:  feature  |      Status:  new         
 Priority:  minor    |   Milestone:  2.11 Release
Component:  general  |     Version:  2.10        
 Keywords:           |       State:              
---------------------+------------------------------------------------------
 We have a number of circular dependencies in the library.  The current
 toposort method isn't flexible enough to handle these.  In most places
 where we have circular dependencies, the order that files are executed
 doesn't matter.  A more tolerant toposort method could resolve
 dependencies even in circular cases - imposing an arbitrary order.  Change
 r10979 is an example of a simplified (and more tolerant) toposort method.

 If we have a more tolerant toposort, then we can move dependency
 declaration into the library source files.  Doing this would remove the
 need to force a specific order in a "first" section of a build
 configuration.

 The one place where we have a circular dependency that causes trouble is
 with Class -> Util -> BaseTypes -> Class.  The Class function only needs
 OpenLayers.Util.extend.  An arbitrary sort order will not guarantee that
 Util is loaded before Class.  One solution is to move the extend method
 into the Class source file.

-- 
Ticket URL: <http://trac.openlayers.org/ticket/2992>
OpenLayers <http://openlayers.org/>
A free AJAX map viewer


More information about the Trac mailing list