]> source.dussan.org Git - jquery.git/commitdiff
Change window to global in the most outer IIFE parameters.
authorMichał Gołębiowski <m.goleb@gmail.com>
Sun, 17 Nov 2013 00:06:44 +0000 (01:06 +0100)
committerMichał Gołębiowski <m.goleb@gmail.com>
Sun, 17 Nov 2013 00:06:44 +0000 (01:06 +0100)
In the most outer IIFE it’s not yet known if the global is window or not.
Using the window variable to denote the global was misleading in that case,
especially that the code didn’t make such assumption, requiring to provide
a Web-like window separately. Renaming window to global clears the confusion.

src/intro.js

index 4b2f4517199bf329441e607b842e222322204fc0..417c61aab7eaffa2bd37dba703a7699ef5ac64a6 100644 (file)
@@ -12,7 +12,7 @@
  * Date: @DATE
  */
 
-(function( window, factory ) {
+(function( global, factory ) {
 
        if ( typeof module === "object" && typeof module.exports === "object" ) {
                // For CommonJS and CommonJS-like environments where a proper window is present,
                // This accentuates the need for the creation of a real window
                // e.g. var jQuery = require("jquery")(window);
                // See ticket #14549 for more info
-               module.exports = window.document ?
-                       factory( window ) :
+               module.exports = global.document ?
+                       factory( global ) :
                        function( w ) {
                                if ( !w.document ) {
-                                       throw new Error("jQuery requires a window with a document");
+                                       throw new Error( "jQuery requires a window with a document" );
                                }
                                return factory( w );
                        };
        } else {
-               factory( window );
+               factory( global );
        }
 
 // Pass this, window may not be defined yet