@@ -214,32 +214,31 @@ authors:
var i,l,item,children;
if(!isXML){
- getById: if (id){
+ if (id && node.getElementById){
// if node == document then we don't need to use contains
- if (!node.getElementById) break getById;
item = node.getElementById(id);
- if (!item || item.id != id) break getById;
- this.push(item, tag, null, parts);
- return;
+ if (item && item.id == id){
+ this.push(item, tag, null, parts);
+ return;
+ }
}
- getById: if (id){
- if (!this.document.getElementById) break getById;
+ if (id && this.document.getElementById){
item = this.document.getElementById(id);
- if (!item || item.id != id) break getById;
- if (!this.contains(node.documentElement||node, item)) break getById;
- this.push(item, tag, null, parts);
- return;
+ if (item && item.id == id && this.contains(node.documentElement || node, item)){
+ this.push(item, tag, null, parts);
+ return;
+ }
}
- getByClass: if (node.getElementsByClassName && classes && !this.cachedGetElementsByClassName){
+ if (node.getElementsByClassName && classes && !this.cachedGetElementsByClassName){
children = node.getElementsByClassName(classes.join(' '));
- if (!(children && children.length)) break getByClass;
- for (i = 0, l = children.length; i < l; i++) this.push(children[i], tag, id, parts, false);
- return;
+ if (children && children.length){
+ for (i = 0, l = children.length; i < l; i++) this.push(children[i], tag, id, parts, false);
+ return;
+ }
}
}
- getByTag: {
- children = local.getByTagName(node, tag);
- if (!(children && children.length)) break getByTag;
+ children = local.getByTagName(node, tag);
+ if (children && children.length){
for (i = 0, l = children.length; i < l; i++) this.push(children[i], null, id, parts);
}
},
@@ -629,7 +628,7 @@ authors:
var parsed, separatorIndex, combinatorIndex, partIndex, reversed, cache = {}, reverseCache = {};
- var parse = function(expression, isReversed){
+ var parse = function(expression, isReversed){
reversed = !!isReversed;
var currentCache = (reversed) ? reverseCache : cache;
if (currentCache[expression]) return currentCache[expression];
slick.js
77837c171b089cd2bada03396c5d145ba92e6c00
Jan Kassens
jan@kassens.net
http://github.com/kassens/slick/commit/7890c763527c33c1427e5705b425af0c4f86bc42
7890c763527c33c1427e5705b425af0c4f86bc42
2009-11-07T14:38:23-08:00
2009-11-07T14:38:23-08:00
replacing breaks with ifs for easier readability at same performance
111f9ffff20a08bd93f296b178823165e3f064dc
Jan Kassens
jan@kassens.net