]> source.dussan.org Git - jquery.git/commitdiff
use for loop instead of jQuery.each in getWH()
authorMike Sherov <mike.sherov@gmail.com>
Fri, 11 Nov 2011 02:23:39 +0000 (21:23 -0500)
committerDave Methvin <dave.methvin@gmail.com>
Fri, 11 Nov 2011 02:59:45 +0000 (21:59 -0500)
src/css.js

index 988a274f60bddf57a3efff837b34b55da2d54279..d31543ea199501a42353800ffef34a9179a69fd7 100644 (file)
@@ -327,20 +327,22 @@ function getWH( elem, name, extra ) {
 
        // Start with offset property
        var val = name === "width" ? elem.offsetWidth : elem.offsetHeight,
-               which = name === "width" ? cssWidth : cssHeight;
+               which = name === "width" ? cssWidth : cssHeight,
+               i = 0,
+               len = which.length;
 
        if ( val > 0 ) {
                if ( extra !== "border" ) {
-                       jQuery.each( which, function() {
+                       for ( ; i < len; i++ ) {
                                if ( !extra ) {
-                                       val -= parseFloat( jQuery.css( elem, "padding" + this ) ) || 0;
+                                       val -= parseFloat( jQuery.css( elem, "padding" + which[ i ] ) ) || 0;
                                }
                                if ( extra === "margin" ) {
-                                       val += parseFloat( jQuery.css( elem, extra + this ) ) || 0;
+                                       val += parseFloat( jQuery.css( elem, extra + which[ i ] ) ) || 0;
                                } else {
-                                       val -= parseFloat( jQuery.css( elem, "border" + this + "Width" ) ) || 0;
+                                       val -= parseFloat( jQuery.css( elem, "border" + which[ i ] + "Width" ) ) || 0;
                                }
-                       });
+                       }
                }
 
                return val + "px";
@@ -356,15 +358,15 @@ function getWH( elem, name, extra ) {
 
        // Add padding, border, margin
        if ( extra ) {
-               jQuery.each( which, function() {
-                       val += parseFloat( jQuery.css( elem, "padding" + this ) ) || 0;
+               for ( ; i < len; i++ ) {
+                       val += parseFloat( jQuery.css( elem, "padding" + which[ i ] ) ) || 0;
                        if ( extra !== "padding" ) {
-                               val += parseFloat( jQuery.css( elem, "border" + this + "Width" ) ) || 0;
+                               val += parseFloat( jQuery.css( elem, "border" + which[ i ] + "Width" ) ) || 0;
                        }
                        if ( extra === "margin" ) {
-                               val += parseFloat( jQuery.css( elem, extra + this ) ) || 0;
+                               val += parseFloat( jQuery.css( elem, extra + which[ i ] ) ) || 0;
                        }
-               });
+               }
        }
 
        return val + "px";