rollup 0.37.1 → 0.37.2

package/dist/rollup.js

@@ -1,6 +1,6 @@
 /*
-	Rollup.js v0.37.1
-	Tue Dec 20 2016 20:08:52 GMT-0500 (EST) - commit b3b10986cc181511d681a80c3535bfd644b986f9
+	Rollup.js v0.37.2
+	Tue Dec 20 2016 23:13:07 GMT-0500 (EST) - commit 37a6869ef967e0bed8fd20dfc70f86dfdb993804
 
 
 	https://github.com/rollup/rollup
@@ -5775,7 +5775,43 @@ simdTypes.forEach( function (t) {
 
 var currentlyCalling = new Set();
 
-function fnHasEffects ( fn ) {
+function isES5Function ( node ) {
+	return node.type === 'FunctionExpression' || node.type === 'FunctionDeclaration';
+}
+
+function hasEffectsNew ( node, scope ) {
+	var inner = node;
+
+	if ( inner.type === 'ExpressionStatement' ) {
+		inner = inner.expression;
+
+		if ( inner.type === 'AssignmentExpression' ) {
+			if ( inner.right.hasEffects( scope ) ) {
+				return true;
+
+			} else {
+				inner = inner.left;
+
+				if ( inner.type === 'MemberExpression' ) {
+					if ( inner.computed && inner.property.hasEffects( scope ) ) {
+						return true;
+
+					} else {
+						inner = inner.object;
+
+						if ( inner.type === 'ThisExpression' ) {
+							return false;
+						}
+					}
+				}
+			}
+		}
+	}
+
+	return node.hasEffects( scope );
+}
+
+function fnHasEffects ( fn, isNew ) {
 	if ( currentlyCalling.has( fn ) ) return false; // prevent infinite loops... TODO there must be a better way
 	currentlyCalling.add( fn );
 
@@ -5784,7 +5820,7 @@ function fnHasEffects ( fn ) {
 	var body = fn.body.type === 'BlockStatement' ? fn.body.body : [ fn.body ];
 
 	for ( var node of body ) {
-		if ( node.hasEffects( scope ) ) {
+		if ( isNew ? hasEffectsNew( node, scope ) : node.hasEffects( scope ) ) {
 			currentlyCalling.delete( fn );
 			return true;
 		}
@@ -5794,14 +5830,14 @@ function fnHasEffects ( fn ) {
 	return false;
 }
 
-function callHasEffects ( scope, callee ) {
+function callHasEffects ( scope, callee, isNew ) {
 	var values = new Set([ callee ]);
 
 	for ( var node of values ) {
 		if ( node === UNKNOWN ) return true; // err on side of caution
 
 		if ( /Function/.test( node.type ) ) {
-			if ( fnHasEffects( node ) ) return true;
+			if ( fnHasEffects( node, isNew && isES5Function( node ) ) ) return true;
 		}
 
 		else if ( isReference( node ) ) {
@@ -5867,7 +5903,7 @@ var CallExpression = (function (Node) {
 	};
 
 	CallExpression.prototype.hasEffects = function hasEffects ( scope ) {
-		return callHasEffects( scope, this.callee );
+		return callHasEffects( scope, this.callee, false );
 	};
 
 	CallExpression.prototype.initialise = function initialise ( scope ) {
@@ -6794,7 +6830,7 @@ var NewExpression = (function (Node) {
 	NewExpression.prototype.constructor = NewExpression;
 
 	NewExpression.prototype.hasEffects = function hasEffects ( scope ) {
-		return callHasEffects( scope, this.callee );
+		return callHasEffects( scope, this.callee, true );
 	};
 
 	return NewExpression;
@@ -7035,6 +7071,16 @@ var VariableDeclarator = (function (Node) {
 		this.activated = true;
 
 		this.run( this.findScope() );
+
+		// if declaration is inside a block, ensure that the block
+		// is marked for inclusion
+		if ( this.parent.kind === 'var' ) {
+			var node = this.parent.parent;
+			while ( /Statement/.test( node.type ) ) {
+				node.shouldInclude = true;
+				node = node.parent;
+			}
+		}
 	};
 
 	VariableDeclarator.prototype.hasEffects = function hasEffects ( scope ) {
@@ -7435,7 +7481,7 @@ var Module = function Module (ref) {
 	});
 
 	// remove existing sourceMappingURL comments
-	var pattern = new RegExp( ("\\/\\/#\\s+" + SOURCEMAPPING_URL$1 + "=.+\\n?"), 'g' );
+	var pattern = new RegExp( ("^\\/\\/#\\s+" + SOURCEMAPPING_URL$1 + "=.+\\n?"), 'gm' );
 	var match;
 	while ( match = pattern.exec( code ) ) {
 		this$1.magicString.remove( match.index, match.index + match[0].length );
@@ -7470,7 +7516,7 @@ Module.prototype.addExport = function addExport ( node ) {
 		}
 
 		else {
-				node.specifiers.forEach( function (specifier) {
+			node.specifiers.forEach( function (specifier) {
 				var name = specifier.exported.name;
 
 				if ( this$1.exports[ name ] || this$1.reexports[ name ] ) {
@@ -7883,6 +7929,7 @@ function amd ( bundle, magicString, ref, options ) {
 	var exportMode = ref.exportMode;
 	var indentString = ref.indentString;
 	var intro = ref.intro;
+	var outro = ref.outro;
 
 	var deps = bundle.externalModules.map( quotePath );
 	var args = bundle.externalModules.map( getName );
@@ -7908,7 +7955,7 @@ function amd ( bundle, magicString, ref, options ) {
 	var exportBlock = getExportBlock( bundle.entryModule, exportMode );
 	if ( exportBlock ) magicString.append( '\n\n' + exportBlock );
 	if ( exportMode === 'named' && options.legacy !== true ) magicString.append( ("\n\n" + esModuleExport) );
-	if ( options.outro ) magicString.append( ("\n" + (options.outro)) );
+	if ( outro ) magicString.append( outro );
 
 	return magicString
 		.indent( indentString )
@@ -7919,6 +7966,7 @@ function amd ( bundle, magicString, ref, options ) {
 function cjs ( bundle, magicString, ref, options ) {
 	var exportMode = ref.exportMode;
 	var intro = ref.intro;
+	var outro = ref.outro;
 
 	intro = ( options.useStrict === false ? intro : ("'use strict';\n\n" + intro) ) +
 	        ( exportMode === 'named' && options.legacy !== true ? (esModuleExport + "\n\n") : '' );
@@ -7963,7 +8011,7 @@ function cjs ( bundle, magicString, ref, options ) {
 
 	var exportBlock = getExportBlock( bundle.entryModule, exportMode, 'module.exports =' );
 	if ( exportBlock ) magicString.append( '\n\n' + exportBlock );
-	if ( options.outro ) magicString.append( ("\n" + (options.outro)) );
+	if ( outro ) magicString.append( outro );
 
 	return magicString;
 }
@@ -7972,8 +8020,9 @@ function notDefault ( name ) {
 	return name !== 'default';
 }
 
-function es ( bundle, magicString, ref, options ) {
+function es ( bundle, magicString, ref ) {
 	var intro = ref.intro;
+	var outro = ref.outro;
 
 	var importBlock = bundle.externalModules
 		.map( function (module) {
@@ -8040,7 +8089,7 @@ function es ( bundle, magicString, ref, options ) {
 	}
 
 	if ( exportBlock ) magicString.append( '\n\n' + exportBlock.trim() );
-	if ( options.outro ) magicString.append( ("\n" + (options.outro)) );
+	if ( outro ) magicString.append( outro );
 
 	return magicString.trim();
 }
@@ -8093,6 +8142,7 @@ function iife ( bundle, magicString, ref, options ) {
 	var exportMode = ref.exportMode;
 	var indentString = ref.indentString;
 	var intro = ref.intro;
+	var outro = ref.outro;
 
 	var globalNameMaker = getGlobalNameMaker( options.globals || blank(), bundle.onwarn );
 
@@ -8133,7 +8183,7 @@ function iife ( bundle, magicString, ref, options ) {
 
 	var exportBlock = getExportBlock( bundle.entryModule, exportMode );
 	if ( exportBlock ) magicString.append( '\n\n' + exportBlock );
-	if ( options.outro ) magicString.append( ("\n" + (options.outro)) );
+	if ( outro ) magicString.append( outro );
 
 	return magicString
 		.indent( indentString )
@@ -8164,6 +8214,7 @@ function umd ( bundle, magicString, ref, options ) {
 	var exportMode = ref.exportMode;
 	var indentString = ref.indentString;
 	var intro = ref.intro;
+	var outro = ref.outro;
 
 	if ( exportMode !== 'none' && !options.moduleName ) {
 		throw new Error( 'You must supply options.moduleName for UMD bundles' );
@@ -8209,7 +8260,7 @@ function umd ( bundle, magicString, ref, options ) {
 	var exportBlock = getExportBlock( bundle.entryModule, exportMode );
 	if ( exportBlock ) magicString.append( '\n\n' + exportBlock );
 	if ( exportMode === 'named' && options.legacy !== true ) magicString.append( ("\n\n" + esModuleExport) );
-	if ( options.outro ) magicString.append( ("\n" + (options.outro)) );
+	if ( outro ) magicString.append( outro );
 
 	return magicString
 		.trim()
@@ -9012,7 +9063,16 @@ Bundle.prototype.render = function render ( options ) {
 		.filter( Boolean )
 		.join( '\n\n' );
 
-	if ( intro ) intro += '\n';
+	if ( intro ) intro += '\n\n';
+
+	var outro = [ options.outro ]
+		.concat(
+			this.plugins.map( function (plugin) { return plugin.outro && plugin.outro(); } )
+		)
+		.filter( Boolean )
+		.join( '\n\n' );
+
+	if ( outro ) outro = "\n\n" + outro;
 
 	var indentString = getIndentString( magicString, options );
 
@@ -9021,7 +9081,7 @@ Bundle.prototype.render = function render ( options ) {
 
 	timeStart( 'render format' );
 
-	magicString = finalise( this, magicString.trim(), { exportMode: exportMode, indentString: indentString, intro: intro }, options );
+	magicString = finalise( this, magicString.trim(), { exportMode: exportMode, indentString: indentString, intro: intro, outro: outro }, options );
 
 	timeEnd( 'render format' );
 
@@ -9045,7 +9105,7 @@ Bundle.prototype.render = function render ( options ) {
 	var bundleSourcemapChain = [];
 
 	code = transformBundle( code, this.plugins, bundleSourcemapChain, options )
-		.replace( new RegExp( ("\\/\\/#\\s+" + SOURCEMAPPING_URL$1 + "=.+\\n?"), 'g' ), '' );
+		.replace( new RegExp( ("^\\/\\/#\\s+" + SOURCEMAPPING_URL$1 + "=.+\\n?"), 'gm' ), '' );
 
 	if ( options.sourceMap ) {
 		timeStart( 'sourceMap' );
@@ -9072,7 +9132,7 @@ Bundle.prototype.render = function render ( options ) {
 	return { code: code, map: map };
 };
 
-	Bundle.prototype.sort = function sort () {
+Bundle.prototype.sort = function sort () {
 		var this$1 = this;
 
 	var hasCycles;
@@ -9148,7 +9208,7 @@ Bundle.prototype.render = function render ( options ) {
 
 					this$1.onwarn(
 						("Module " + (a.id) + " may be unable to evaluate without " + (b.id) + ", but is included first due to a cyclical dependency. Consider swapping the import statements in " + parent + " to ensure correct ordering")
-					);
+						);
 				}
 			};
 
@@ -9159,7 +9219,7 @@ Bundle.prototype.render = function render ( options ) {
 	return ordered;
 };
 
-var VERSION = '0.37.1';
+var VERSION = '0.37.2';
 
 var ALLOWED_KEYS = [
 	'acorn',