Opened 16 years ago
Closed 15 years ago
#1731 closed bug (fixed)
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.
Attachments (2)
Change History (4)
comment:1 Changed 16 years ago by
Changed 15 years ago by
Attachment: | 1731.2.diff added |
---|
Fixes leaks in remove, html, empty and expands small variables
Note: See
TracTickets for help on using
tickets.
Patch fixed unbind issue in ie6.