Opened 13 years ago
Closed 12 years ago
#6677 closed bug (fixed)
Missing var declarations
Reported by: | msamuel | Owned by: | |
---|---|---|---|
Priority: | undecided | Milestone: | 1.4.3 |
Component: | ajax | Version: | 1.4.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
The attached diff from version 1.4.3pre shows some missing var declarations and a buggy exception handler.
It was derived by running dist/jquery.js
Attachments (1)
Change History (5)
Changed 13 years ago by
Attachment: | 1.4.3.pre_dist_jquery_diff.txt added |
---|
comment:1 Changed 13 years ago by
It was derived by running dist/jquery.js through http://caja.appspot.com/tools/
comment:2 Changed 13 years ago by
need: | Review → Commit |
---|
Some of those are okay, the variables are declared upfunction. If they are declared with "var" in both places, jslint will whine. I tend to agree with caja and declare it var everywhere in my own code.
Fixes for the two bugs:
http://github.com/dmethvin/jquery/commit/0a1a8605a040c0f7aedb1df6bb4942383f4550b4
http://github.com/dmethvin/jquery/commit/ab89a1b2cff4b02b0ce6e5ca35aed4fbd2c478d5
comment:3 Changed 13 years ago by
Component: | unfiled → ajax |
---|
comment:4 Changed 12 years ago by
Priority: | → undecided |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Diff after running linter over dist/jquery.js