Skip to content

Commit

Permalink
lint fixes and removing unused / duplicate variables, refs #13036 !st…
Browse files Browse the repository at this point in the history
…rict

git-svn-id: http://svn.dojotoolkit.org/src/dojo/trunk@26141 560b804f-0ae3-0310-86f3-f6aa0a117693
  • Loading branch information
wkeese committed Aug 10, 2011
1 parent 0109611 commit 9e976f6
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 11 deletions.
13 changes: 6 additions & 7 deletions _base/NodeList.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,16 +47,15 @@ var NodeList = query.NodeList;
var NodeList = dojo.NodeList;
=====*/
var nl = NodeList,
nlp = nl.prototype;
var nlp = NodeList.prototype;

// don't bind early to dojo.connect since we no longer explicitly depend on it
nlp.connect = nl._adaptAsForEach(function(){
nlp.connect = NodeList._adaptAsForEach(function(){
return dojo.connect.apply(this, arguments);
});
nlp.coords = nl._adaptAsMap(dojo.coords);
nlp.coords = NodeList._adaptAsMap(dojo.coords);

nl.events = [
NodeList.events = [
// summary:
// list of all DOM events used in NodeList
"blur", "focus", "change", "click", "error", "keydown", "keypress",
Expand All @@ -67,7 +66,7 @@ var NodeList = query.NodeList;
// FIXME: pseudo-doc the above automatically generated on-event functions

// syntactic sugar for DOM events
array.forEach(nl.events, function(evt){
array.forEach(NodeList.events, function(evt){
var _oe = "on" + evt;
nlp[_oe] = function(a, b){
return this.connect(_oe, a, b);
Expand Down Expand Up @@ -96,6 +95,6 @@ var NodeList = query.NodeList;
}
);

dojo.NodeList = nl;
dojo.NodeList = NodeList;
return dojo.NodeList;
});
2 changes: 1 addition & 1 deletion _base/connect.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ function connect_(obj, event, context, method, dontFix){
}
}
return on(obj, event, lang.hitch(context, method), dontFix);
};
}

var _punctMap = {
106:42,
Expand Down
4 changes: 2 additions & 2 deletions _base/lang.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
define(["./kernel", "../has", "./sniff"], function(dojo, has, sniff){
define(["./kernel", "../has", "./sniff"], function(dojo, has){
// module:
// dojo/_base/lang
// summary:
Expand Down Expand Up @@ -210,7 +210,7 @@ define(["./kernel", "../has", "./sniff"], function(dojo, has, sniff){
// RegExp
return new RegExp(src); // RegExp
}
var r, i, l, s, name;
var r, i, l;
if(lang.isArray(src)){
// array
r = [];
Expand Down
2 changes: 1 addition & 1 deletion _base/loader.js
Original file line number Diff line number Diff line change
Expand Up @@ -611,7 +611,7 @@ define(["./kernel", "../has", "require", "module", "./json", "./lang", "./array"
execQ.push(module);
}
return undefined;
};
}

var result = doRequire(moduleName);
if(has("config-publishRequireResult") && !lang.exists(moduleName) && result!==undefined){
Expand Down

0 comments on commit 9e976f6

Please sign in to comment.