Changeset 5479

Timestamp:

05/06/08 22:36:30 (8 months ago)

Author:

aflesler

Message:

jquery core: removing needless 'else' and 'new' in $.fn.init.

Files:

• trunk/jquery/src/core.js (modified) (4 diffs)

trunk/jquery/src/core.js

@@ -37 +37 @@
             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 );
@@ -55 +55 @@
 
                     // Make sure an element was located
-                    if ( elem )
+                    if ( elem ){
                         // Handle the case where IE and Opera return items
                         // by name instead of ID
@@ -62 +62 @@
 
                         // 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 = [];
                 }
 
@@ -75 +70 @@
             // (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));