Skip to main content

Bug Tracker

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.

Related tickets are #1697, #1618 and #1610.

Attachments (2)
  • 1731.2.diff (6.5 KB) - added by brandon October 05, 2007 10:32PM UTC.

    Fixes leaks in remove, html, empty and expands small variables

  • 1731.diff (1.2 KB) - added by brandon October 01, 2007 01:24PM UTC.

    Patch

Change History (2)

Changed September 28, 2007 04:29PM UTC by harking comment:1

Patch fixed unbind issue in ie6.

Changed October 06, 2007 05:06PM UTC by brandon comment:2

resolution: → fixed
status: newclosed

Fixed in Rev [3578]