OpenLayers OpenLayers

Ticket #369 (new task)

Opened 2 years ago

Last modified 3 weeks ago

Thorough Review of OpenLayers Memory Management

Reported by: euzuro Assigned to: euzuro sde
Priority: major Milestone: Maintenance
Component: general Version:
Keywords: Cc:
State:

Description

Need to sift through the code and make sure all object creation and deletions are in order, as well as event registerings and unregisterings.

Specifically, for every time a call is made to new, there should be a corresponding call to the object's destroy().

same goes for event.register() and event.unregister()

also we should consider adding a function removeFromMap() that would be the counter to setMap(), which would be the convenient place to register/unregister map events.

Change History

12/06/06 16:05:19 changed by euzuro

  • milestone set to 2.3 Release.

12/06/06 16:43:56 changed by euzuro

  • owner set to euzuro sde.

12/28/06 20:13:20 changed by euzuro

  • milestone changed from 2.3 Release to 2.4 Release.

02/19/07 20:58:28 changed by openlayers

  • keywords set to review.

patch added as per http://www.nabble.com/Markers%27-events-memory-leak--tf3232599.html

stopObservingElement method added to Event object; stopObservingElement call added to Icon's destroy method;

03/08/07 13:03:17 changed by sderle

  • milestone changed from 2.4 Release to 2.5 Release.

03/31/07 16:47:33 changed by crschmidt

  • keywords deleted.

06/29/07 16:28:41 changed by euzuro

  • milestone changed from 2.5 Release to Maintenance.

11/07/07 14:35:26 changed by camerons

To search for memory leaks, Drip is reported to be useful. http://outofhanwell.com/ieleak/index.php?title=Main_Page

09/17/08 05:54:12 changed by openlayers

  • state changed.

sIEve is an extended version of Drip.
Can be found here: http://home.wanadoo.nl/jsrosman/