Skip to content

Commit

Permalink
jquery core: removing needless 'else' and 'new' in $.fn.init.
Browse files Browse the repository at this point in the history
  • Loading branch information
flesler committed May 6, 2008
1 parent 46931d3 commit 69d9abf
Showing 1 changed file with 8 additions and 13 deletions.
21 changes: 8 additions & 13 deletions src/core.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ jQuery.fn = jQuery.prototype = {
this[0] = selector;
this.length = 1;
return this;

}
// Handle HTML strings
} else if ( typeof selector == "string" ) {
if ( typeof selector == "string" ) {
// Are we dealing with HTML string or an ID?
var match = quickExpr.exec( selector );

Expand All @@ -54,32 +54,27 @@ jQuery.fn = jQuery.prototype = {
var elem = document.getElementById( match[3] );

// Make sure an element was located
if ( elem )
if ( elem ){
// Handle the case where IE and Opera return items
// by name instead of ID
if ( elem.id != match[3] )
return jQuery().find( selector );

// Otherwise, we inject the element directly into the jQuery object
else {
this[0] = elem;
this.length = 1;
return this;
}

else
selector = [];
return jQuery( elem );
}
selector = [];
}

// HANDLE: $(expr, [context])
// (which is just equivalent to: $(content).find(expr)
} else
return new jQuery( context ).find( selector );
return jQuery( context ).find( selector );

// HANDLE: $(function)
// Shortcut for document ready
} else if ( jQuery.isFunction( selector ) )
return new jQuery( document )[ jQuery.fn.ready ? "ready" : "load" ]( selector );
return jQuery( document )[ jQuery.fn.ready ? "ready" : "load" ]( selector );

return this.setArray(jQuery.makeArray(selector));
},
Expand Down

0 comments on commit 69d9abf

Please sign in to comment.