#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)
Change History (7)
comment:1 Changed 15 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Changed 15 years ago by
Attachment: | warn_fix1.diff added |
---|
Not very complex fix for the warnings on init
comment:2 Changed 15 years ago by
Resolution: | wontfix |
---|---|
Status: | closed → reopened |
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 15 years ago by
Resolution: | → wontfix |
---|---|
Status: | reopened → closed |
You can't just check for elem.length as that will be false when it is === 0.
comment:4 Changed 15 years ago by
Resolution: | wontfix |
---|---|
Status: | closed → reopened |
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 15 years ago by
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 15 years ago by
Resolution: | → wontfix |
---|---|
Status: | reopened → closed |
Resolving those makes the code a lot more complex without actually solving any problems.