feat(moduletemplate): Wrap function module template wrappers with parens

btmills · btmills · commit 2def8b4ba533 · 2016-12-31T11:47:49.000-05:00
Back-ports c7b06ad (webpack#3658) to webpack-1 based on https://twitter.com/TheLarkInn/status/815230804729196544.
diff --git a/lib/FunctionModuleTemplatePlugin.js b/lib/FunctionModuleTemplatePlugin.js
@@ -15,9 +15,9 @@ FunctionModuleTemplatePlugin.prototype.apply = function(moduleTemplate) {
 		if((module.arguments && module.arguments.length !== 0) || module.hasDependencies()) {
 			defaultArguments.push("__webpack_require__");
 		}
-		source.add("/***/ function(" + defaultArguments.concat(module.arguments || []).join(", ") + ") {\n\n");
+		source.add("/***/ (function(" + defaultArguments.concat(module.arguments || []).join(", ") + ") {\n\n");
 		source.add(new PrefixSource(this.outputOptions.sourcePrefix, moduleSource));
-		source.add("\n\n/***/ }");
+		source.add("\n\n/***/ })");
 		return source;
 	});
 	moduleTemplate.plugin("package", function(moduleSource, module) {
diff --git a/test/statsCases/chunks/expected.txt b/test/statsCases/chunks/expected.txt
@@ -2,9 +2,9 @@ Hash: 27b51279f30c19c26769
 Time: Xms
       Asset       Size  Chunks             Chunk Names
   bundle.js    3.83 kB       0  [emitted]  main
-1.bundle.js  159 bytes       1  [emitted]  
-2.bundle.js  101 bytes       2  [emitted]  
-3.bundle.js  180 bytes       3  [emitted]  
+1.bundle.js  161 bytes       1  [emitted]  
+2.bundle.js  103 bytes       2  [emitted]  
+3.bundle.js  184 bytes       3  [emitted]  
 chunk    {0} bundle.js (main) 73 bytes [rendered]
     > main [0] (webpack)/test/statsCases/chunks/index.js 
     [0] (webpack)/test/statsCases/chunks/index.js 51 bytes {0} [built]
diff --git a/test/statsCases/define-plugin/expected.txt b/test/statsCases/define-plugin/expected.txt
@@ -3,13 +3,13 @@ Child
     Hash: 22655fdde3386763624e
     Time: Xms
         Asset     Size  Chunks             Chunk Names
-    bundle.js  1.43 kB       0  [emitted]  main
+    bundle.js  1.44 kB       0  [emitted]  main
     chunk    {0} bundle.js (main) 24 bytes [rendered]
         [0] (webpack)/test/statsCases/define-plugin/index.js 24 bytes {0} [built]
 Child
     Hash: b84e3c3c9ea7106aac84
     Time: Xms
         Asset     Size  Chunks             Chunk Names
-    bundle.js  1.43 kB       0  [emitted]  main
+    bundle.js  1.44 kB       0  [emitted]  main
     chunk    {0} bundle.js (main) 24 bytes [rendered]
         [0] (webpack)/test/statsCases/define-plugin/index.js 24 bytes {0} [built]
diff --git a/test/statsCases/preset-verbose/expected.txt b/test/statsCases/preset-verbose/expected.txt
@@ -2,9 +2,9 @@ Hash: 27b51279f30c19c26769
 Time: Xms
       Asset       Size  Chunks             Chunk Names
   bundle.js    3.83 kB       0  [emitted]  main
-1.bundle.js  159 bytes       1  [emitted]  
-2.bundle.js  101 bytes       2  [emitted]  
-3.bundle.js  180 bytes       3  [emitted]  
+1.bundle.js  161 bytes       1  [emitted]  
+2.bundle.js  103 bytes       2  [emitted]  
+3.bundle.js  184 bytes       3  [emitted]  
 chunk    {0} bundle.js (main) 73 bytes [rendered]
     [0] (webpack)/test/statsCases/preset-verbose/index.js 51 bytes {0} [built]
        factory:Xms building:Xms = Xms
