No longer use arguments.callee or RegExp (use new RegExp, instead) for ES 3.1 and...
authorJohn Resig <jeresig@gmail.com>
Thu, 26 Feb 2009 18:00:41 +0000 (18:00 +0000)
committerJohn Resig <jeresig@gmail.com>
Thu, 26 Feb 2009 18:00:41 +0000 (18:00 +0000)
src/event.js
src/selector.js
src/support.js

index 230af19..26c5620 100644 (file)
@@ -116,7 +116,7 @@ jQuery.event = {
                                        // Namespaced event handlers
                                        var namespaces = type.split(".");
                                        type = namespaces.shift();
-                                       var namespace = RegExp("(^|\\.)" + namespaces.slice().sort().join(".*\\.") + "(\\.|$)");
+                                       var namespace = new RegExp("(^|\\.)" + namespaces.slice().sort().join(".*\\.") + "(\\.|$)");
 
                                        if ( events[type] ) {
                                                // remove the given handler for the given type
@@ -249,7 +249,7 @@ jQuery.event = {
                // Cache this now, all = true means, any handler
                all = !namespaces.length && !event.exclusive;
                
-               var namespace = RegExp("(^|\\.)" + namespaces.slice().sort().join(".*\\.") + "(\\.|$)");
+               var namespace = new RegExp("(^|\\.)" + namespaces.slice().sort().join(".*\\.") + "(\\.|$)");
 
                handlers = ( jQuery.data(this, "events") || {} )[event.type];
 
@@ -354,7 +354,7 @@ jQuery.event = {
                        },
                        teardown:  function( namespaces ){
                                if ( namespaces.length ) {
-                                       var remove = 0, name = RegExp("(^|\\.)" + namespaces[0] + "(\\.|$)");
+                                       var remove = 0, name = new RegExp("(^|\\.)" + namespaces[0] + "(\\.|$)");
                                        
                                        jQuery.each( (jQuery.data(this, "events").live || {}), function(){
                                                if ( name.test(this.type) )
@@ -560,7 +560,7 @@ jQuery.fn.extend({
 });
 
 function liveHandler( event ){
-       var check = RegExp("(^|\\.)" + event.type + "(\\.|$)"),
+       var check = new RegExp("(^|\\.)" + event.type + "(\\.|$)"),
                stop = true,
                elems = [];
 
index e58b257..3bfc5f5 100644 (file)
@@ -659,7 +659,7 @@ var Expr = Sizzle.selectors = {
 var origPOS = Expr.match.POS;
 
 for ( var type in Expr.match ) {
-       Expr.match[ type ] = RegExp( Expr.match[ type ].source + /(?![^\[]*\])(?![^\(]*\))/.source );
+       Expr.match[ type ] = new RegExp( Expr.match[ type ].source + /(?![^\[]*\])(?![^\(]*\))/.source );
 }
 
 var makeArray = function(array, results) {
index b11f847..4298250 100644 (file)
        root.removeChild( script );
 
        if ( div.attachEvent && div.fireEvent ) {
-               div.attachEvent("onclick", function(){
+               div.attachEvent("onclick", function click(){
                        // Cloning a node shouldn't copy over any
                        // bound event handlers (IE does this)
                        jQuery.support.noCloneEvent = false;
-                       div.detachEvent("onclick", arguments.callee);
+                       div.detachEvent("onclick", click);
                });
                div.cloneNode(true).fireEvent("onclick");
        }