Bug Tracker

Opened 6 years ago

Closed 6 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:


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

Change History (4)

comment:1 Changed 6 years ago by msamuel

It was derived by running dist/jquery.js through http://caja.appspot.com/tools/

comment:2 Changed 6 years ago by dmethvin

  • need changed from Review to 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:



comment:3 Changed 6 years ago by dmethvin

  • Component changed from unfiled to ajax

comment:4 Changed 6 years ago by john

  • Priority set to undecided
  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.