speed: committing a failing experiment of concat vs slice

[jquery.git] / src / attributes.js

diff --git a/src/attributes.js b/src/attributes.js
index 9e66df0..93e0433 100644 (file)
--- a/src/attributes.js
+++ b/src/attributes.js
@@ -100,15 +100,13 @@ jQuery.fn.extend({
                                return;
 
                        if ( jQuery.isArray(val) && /radio|checkbox/.test( this.type ) )
-                               this.checked = (jQuery.inArray(this.value, val) >= 0 ||
-                                       jQuery.inArray(this.name, val) >= 0);
+                               this.checked = jQuery.inArray(this.value || this.name, val) >= 0;
 
                        else if ( jQuery.nodeName( this, "select" ) ) {
                                var values = jQuery.makeArray(val);
 
                                jQuery( "option", this ).each(function(){
-                                       this.selected = (jQuery.inArray( this.value, values ) >= 0 ||
-                                               jQuery.inArray( this.text, values ) >= 0);
+                                       this.selected = jQuery.inArray( this.value || this.text, values ) >= 0;
                                });
 
                                if ( !values.length )
@@ -192,7 +190,7 @@ jQuery.extend({
                if (!elem || elem.nodeType == 3 || elem.nodeType == 8)
                        return undefined;
 
-               var notxml = !elem.tagName || !jQuery.isXMLDoc( elem ),
+               var notxml = elem.nodeType !== 1 || !jQuery.isXMLDoc( elem ),
                        // Whether we are setting (or getting)
                        set = value !== undefined;
 
@@ -200,7 +198,7 @@ jQuery.extend({
                name = notxml && jQuery.props[ name ] || name;
 
                // Only do all the following if this is a node (faster for style)
-               if ( elem.tagName ) {
+               if ( elem.nodeType === 1 ) {
 
                        // These attributes require special treatment
                        var special = /href|src|style/.test( name );
@@ -214,7 +212,7 @@ jQuery.extend({
                        if ( name in elem && notxml && !special ) {
                                if ( set ){
                                        // We can't allow the type property to be changed (since it causes problems in IE)
-                                       if ( name == "type" && elem.nodeName.match(/(button|input)/i) && elem.parentNode )
+                                       if ( name == "type" && /(button|input)/i.test(elem.nodeName) && elem.parentNode )
                                                throw "type property can't be changed";
 
                                        elem[ name ] = value;
@@ -230,9 +228,9 @@ jQuery.extend({
                                        var attributeNode = elem.getAttributeNode( "tabIndex" );
                                        return attributeNode && attributeNode.specified
                                                ? attributeNode.value
-                                               : elem.nodeName.match(/(button|input|object|select|textarea)/i)
+                                               : /(button|input|object|select|textarea)/i.test(elem.nodeName)
                                                        ? 0
-                                                       : elem.nodeName.match(/^(a|area)$/i) && elem.href
+                                                       : /^(a|area)$/i.test(elem.nodeName) && elem.href
                                                                ? 0
                                                                : undefined;
                                }