Move cases of .replace(re, Function) out from inline (to avoid being redeclared on...
authorJohn Resig <jeresig@gmail.com>
Sun, 19 Jul 2009 15:57:43 +0000 (15:57 +0000)
committerJohn Resig <jeresig@gmail.com>
Sun, 19 Jul 2009 15:57:43 +0000 (15:57 +0000)
src/css.js
src/manipulation.js

index 2395bdf..c32034d 100644 (file)
@@ -3,7 +3,10 @@ var exclude = /z-?index|font-?weight|opacity|zoom|line-?height/i,
        // cache check for defaultView.getComputedStyle
        getComputedStyle = document.defaultView && document.defaultView.getComputedStyle,
        // normalize float css property
-       styleFloat = jQuery.support.cssFloat ? "cssFloat" : "styleFloat";
+       styleFloat = jQuery.support.cssFloat ? "cssFloat" : "styleFloat",
+       fcamelCase = function(all, letter){
+               return letter.toUpperCase();
+       };
 
 jQuery.fn.css = function( name, value ) {
        var options = name, isFunction = jQuery.isFunction( value );
@@ -83,9 +86,7 @@ jQuery.extend({
                if ( /float/i.test( name ) )
                        name = styleFloat;
 
-               name = name.replace(/-([a-z])/ig, function(all, letter){
-                       return letter.toUpperCase();
-               });
+               name = name.replace(/-([a-z])/ig, fcamelCase);
 
                if ( set )
                        style[ name ] = value;
@@ -163,9 +164,7 @@ jQuery.extend({
                                ret = "1";
 
                } else if ( elem.currentStyle ) {
-                       var camelCase = name.replace(/\-(\w)/g, function(all, letter){
-                               return letter.toUpperCase();
-                       });
+                       var camelCase = name.replace(/\-(\w)/g, fcamelCase);
 
                        ret = elem.currentStyle[ name ] || elem.currentStyle[ camelCase ];
 
index 0051bbc..f577cf8 100644 (file)
@@ -4,7 +4,12 @@ var rinlinejQuery = / jQuery\d+="(?:\d+|null)"/g,
        rxhtmlTag = /(<(\w+)[^>]*?)\/>/g,
        rselfClosing = /^(?:abbr|br|col|img|input|link|meta|param|hr|area|embed)$/i,
        rinsideTable = /^<(thead|tbody|tfoot|colg|cap)/,
-       rtbody = /<tbody/i;
+       rtbody = /<tbody/i,
+       fcloseTag = function(all, front, tag){
+               return rselfClosing.test(tag) ?
+                       all :
+                       front + "></" + tag + ">";
+       });
 
 jQuery.fn.extend({
        text: function( text ) {
@@ -303,11 +308,7 @@ jQuery.extend({
                        // Convert html string into DOM nodes
                        if ( typeof elem === "string" ) {
                                // Fix "XHTML"-style tags in all browsers
-                               elem = elem.replace(rxhtmlTag, function(all, front, tag){
-                                       return rselfClosing.test(tag) ?
-                                               all :
-                                               front + "></" + tag + ">";
-                               });
+                               elem = elem.replace(rxhtmlTag, fcloseTag);
 
                                // Trim whitespace, otherwise indexOf won't work as expected
                                var tags = elem.replace(rleadingWhitespace, "")