Modify ↓
Ticket #8203 (closed bug: fixed)
Remove un-needed "someVar = null;"s
| Reported by: | musicisair | Owned by: | snover |
|---|---|---|---|
| Priority: | blocker | Milestone: | 1.6 |
| Component: | misc | Version: | 1.5 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
It looks like jQuery has some overzealous memoryleak someVar = null lines. It can't hurt to clean them up a bit.
A specific example can be found in $.support.scriptEval:
release memory in IE root = script = id = null;
Change History
comment:2 Changed 2 years ago by jitter
- Priority changed from undecided to low
- Component changed from unfiled to misc
comment:3 Changed 2 years ago by snover
- Owner set to snover
- Priority changed from low to blocker
- Status changed from new to assigned
- Milestone changed from 1.next to 1.6
comment:4 Changed 2 years ago by Dave Methvin
- Status changed from assigned to closed
- Resolution set to fixed
Fixes #8203. Remove some misguided nulling of elements done in the name of IE memory cleanup. Based on a patch by davidmurdoch in pull 226.
Changeset: ad4152709cd7c7b7fdafd2dc0b266fd273aff5d5
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.

https://github.com/jquery/jquery/pull/226