jquery core: improves #3248. Functions are supported by isFunction.

[jquery.git] / src / core.js

diff --git a/src/core.js b/src/core.js
index cbc3b40..3a2548f 100644 (file)
--- a/src/core.js
+++ b/src/core.js
@@ -90,9 +90,6 @@ jQuery.fn = jQuery.prototype = {
                return this.length;
        },
 
-       // The number of elements contained in the matched element set
-       length: 0,
-
        // Get the Nth element in the matched element set OR
        // Get the whole matched element set as a clean array
        get: function( num ) {
@@ -615,10 +612,14 @@ jQuery.extend({
        },
 
        // See test/unit/core.js for details concerning this function.
-       // Since 1.3 DOM methods and function like alert
+       // Since version 1.3, DOM methods and functions like alert
        // aren't supported. They return false on IE (#2968).
+       
+       // Memory leaks appear in IE6 when applying instanceof 
+       // to the window, document or any other COM object (#3485)
+       // http://ajaxian.com/archives/working-aroung-the-instanceof-memory-leak
        isFunction: function( fn ) {
-               return fn instanceof Function;
+               return !!fn && !!fn.hasOwnProperty && fn instanceof Function;
        },
 
        // check if an element is in a (or is an) XML document
@@ -1128,13 +1129,15 @@ jQuery.extend({
                return (text || "").replace( /^\s+|\s+$/g, "" );
        },
 
+       // NOTE: Due to the conflict with Scriptaculous (http://dev.jquery.com/ticket/3248)
+       // We remove support for functions since jQuery 1.3
        makeArray: function( array ) {
                var ret = [];
 
                if( array != null ){
                        var i = array.length;
-                       //the window, strings and functions also have 'length'
-                       if( i == null || array.split || array.setInterval || array.call )
+                       // The window, strings (and functions) also have 'length'
+                       if( i == null || typeof array == 'string' || jQuery.isFunction(array) || array.setInterval )
                                ret[0] = array;
                        else
                                while( i )