Simplified module pattern

Xophmeister · Jan 17, 2013 · 8d994c4 · 8d994c4
1 parent 9659f6e
commit 8d994c4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/jsSelecta.js b/jsSelecta.js
@@ -2,7 +2,7 @@
 // Christopher Harrison (c) 2013
 // MIT License
 
-(function(root) {
+(function(root, undefined) {
   var extend = function(base, child) {
         // Shallow copy
         for (var x in child) {
@@ -142,10 +142,10 @@
   });
 
   // Enter selecta!
-  if (typeof module !== 'undefined' && module.exports) {
+  if (typeof module !== undefined && module.exports) {
     // CommonJS Module (e.g., for node.js)
     module.exports = selecta;
-  } else if (typeof define !== 'undefined' && define.amd) {
+  } else if (typeof define !== undefined && define.amd) {
     // AMD Module (e.g., for RequireJS)
     define('jsSelecta', [], function() { return selecta; });
   } else {
@@ -156,4 +156,4 @@
       root.selecta = selecta;
     }
   }
-})((function() { return this; })()); // i.e., Global object
+})(this);
diff --git a/tdd/test.js b/tdd/test.js
@@ -23,7 +23,7 @@
 
 */
 
-(function(root) {
+(function(root, undefined) {
   var contains = function(a, b) {
         var passed = true;
 
@@ -72,11 +72,11 @@
       };
 
   // Instantiate
-  if (typeof module !== 'undefined' && module.exports) {
+  if (typeof module !== undefined && module.exports) {
     // CommonJS Module (e.g., for node.js)
     var selecta = require('../jsSelecta');
     module.exports = test;
-  } else if (typeof define !== 'undefined' && define.amd) {
+  } else if (typeof define !== undefined && define.amd) {
     // AMD Module (e.g., for RequireJS)
     define('test', ['jsSelecta'], function() { return test; });
   } else {
@@ -87,4 +87,4 @@
       root.test = test;
     }
   }
-})((function() { return this; } )());
+})(this);