Side navigation
#1731 closed bug (fixed)
Opened September 26, 2007 06:21AM UTC
Closed October 06, 2007 05:06PM UTC
New "handler" created for each event added and leaks memory in IE
Reported by: | arrix | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | 1.2.2 |
Component: | event | Version: | 1.2.1 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
See the conversation titled ''possible bug in event code'' in dev list.
In event.js line 46
var handle = jQuery.data(element, "handle", function(){
should instead be
var handle = jQuery.data(element, "handle") || jQuery.data(element, "handle", function () {
David: "It looks like the code was always creating a new handle for each event
added instead of getting an existing handle out of the cache when it
was already there."
The handlers overwritten can never be removed and cause memory leaks.
Patch fixed unbind issue in ie6.