OpenLayers OpenLayers

Ticket #558 (closed bug: fixed)

Opened 2 years ago

Last modified 1 year ago

destroy events objects in objects' destructors

Reported by: euzuro Assigned to:
Priority: major Milestone: 2.4 Release
Component: general Version:
Keywords: Cc:
State:

Description

now that we have created a full on events.destroy() function (#510), we should actually *use* it.

Attachments

destroyEvents.patch (2.3 kB) - added by euzuro on 03/23/07 21:55:04.
in the classes where an events object exists and is instantiated, destroy it on the objects destruction. This is a modifcation of http://trac.openlayers.org/attachment/ticket/510/510-2.patch

Change History

03/23/07 21:55:04 changed by euzuro

  • attachment destroyEvents.patch added.

in the classes where an events object exists and is instantiated, destroy it on the objects destruction. This is a modifcation of http://trac.openlayers.org/attachment/ticket/510/510-2.patch

03/23/07 21:58:07 changed by euzuro

  • keywords set to review.

03/25/07 20:51:25 changed by crschmidt

  • status changed from new to closed.
  • resolution set to fixed.

This patch should fix up a number of small memory leaks in OpenLayers. With some testing of creating a large number of popups, I was able to reproduce what appeared to be less memory usage with this patch applied than before.

03/25/07 20:53:14 changed by crschmidt

r2894.

07/05/07 17:03:37 changed by euzuro

  • keywords deleted.