Bug Tracker

Opened 12 years ago

Closed 12 years ago

Last modified 11 years ago

#2079 closed bug (wontfix)

javascript.options.strict warnings [1.2.2b]

Reported by: Byron Owned by:
Priority: major Milestone: 1.2.2
Component: core Version: 1.2.1
Keywords: Cc:
Blocked by: Blocking:

Description

some new warnings have been introduced in 1.2.2b

When jQuery initializes the following warnings are thrown

reference to undefined property jQuery.cache[id][name]
http://code.jquery.com/jquery-1.2.2b.js
Line 661
reference to undefined property elem.nodeType
http://code.jquery.com/jquery-1.2.2b.js
Line 1793

The Following Warnings are fired when an animation is performed:

reference to undefined property optall.queue
http://code.jquery.com/jquery-1.2.2b.js
Line 2830
reference to undefined property elem.nodeType
http://code.jquery.com/jquery-1.2.2b.js
Line 1011
reference to undefined property opt.queue
http://code.jquery.com/jquery-1.2.2b.js
Line 2985

Attachments (1)

warn_fix1.diff (1.1 KB) - added by Vegar 12 years ago.
Not very complex fix for the warnings on init

Download all attachments as: .zip

Change History (7)

comment:1 Changed 12 years ago by joern

Resolution: wontfix
Status: newclosed

Resolving those makes the code a lot more complex without actually solving any problems.

Changed 12 years ago by Vegar

Attachment: warn_fix1.diff added

Not very complex fix for the warnings on init

comment:2 Changed 12 years ago by Vegar

Resolution: wontfix
Status: closedreopened

I've attached a fix for the init-warnings (that doesn't make the code a lot more complex). And it solves a problem: It makes the javascript console readable when using jQuery.

comment:3 Changed 12 years ago by brandon

Resolution: wontfix
Status: reopenedclosed

You can't just check for elem.length as that will be false when it is === 0.

comment:4 Changed 12 years ago by Byron

Resolution: wontfix
Status: closedreopened

It doesnt makes sense to leave this unfixed, until now, all strict warnings issues have been resolved. http://dev.jquery.com/search?q=strict+warnings whats changed?

comment:5 Changed 12 years ago by davidserduke

What has changed is the "fixes" hurt performance. So far we have decided that isn't worth it. See this thread for more info:

http://groups.google.com/group/jquery-dev/browse_thread/thread/f2926d95fcafb18

comment:6 Changed 12 years ago by brandon

Resolution: wontfix
Status: reopenedclosed
Note: See TracTickets for help on using tickets.