Bug Tracker

Modify

Ticket #2079 (closed bug: wontfix)

Opened 5 years ago

Last modified 5 years ago

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:
Blocking: Blocked by:

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

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

Change History

comment:1 Changed 5 years ago by joern

  • Status changed from new to closed
  • Resolution set to wontfix

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

Changed 5 years ago by Vegar

Not very complex fix for the warnings on init

comment:2 Changed 5 years ago by Vegar

  • Status changed from closed to reopened
  • Resolution wontfix deleted

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 5 years ago by brandon

  • Status changed from reopened to closed
  • Resolution set to wontfix

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

comment:4 Changed 5 years ago by Byron

  • Status changed from closed to reopened
  • Resolution wontfix deleted

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 5 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 5 years ago by brandon

  • Status changed from reopened to closed
  • Resolution set to wontfix

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.