Renamed alwaysCopy to needsUniqueCopy

tinganho · tinganho · commit e9f8a6f701d1 · 2015-05-15T09:18:24.000+08:00
diff --git a/src/compiler/emitter.ts b/src/compiler/emitter.ts
@@ -1366,7 +1366,7 @@ var __param = (this && this.__param) || function (paramIndex, decorator) {
                 return true;
             }
 
-            function emitListWithSpread(elements: Expression[], alwaysCopy: boolean, multiLine: boolean, trailingComma: boolean) {
+            function emitListWithSpread(elements: Expression[], needsUniqueCopy: boolean, multiLine: boolean, trailingComma: boolean) {
                 let pos = 0;
                 let group = 0;
                 let length = elements.length;
@@ -1383,7 +1383,7 @@ var __param = (this && this.__param) || function (paramIndex, decorator) {
                         e = (<SpreadElementExpression>e).expression;
                         emitParenthesizedIf(e, /*parenthesized*/ group === 0 && needsParenthesisForPropertyAccessOrInvocation(e));
                         pos++;
-                        if (pos === length && group === 0 && alwaysCopy && e.kind !== SyntaxKind.ArrayLiteralExpression) {
+                        if (pos === length && group === 0 && needsUniqueCopy && e.kind !== SyntaxKind.ArrayLiteralExpression) {
                             write(".slice()");
                         }
                     }
@@ -1425,7 +1425,7 @@ var __param = (this && this.__param) || function (paramIndex, decorator) {
                     write("]");
                 }
                 else {
-                    emitListWithSpread(elements, /*alwaysCopy*/ true, /*multiLine*/(node.flags & NodeFlags.MultiLine) !== 0,
+                    emitListWithSpread(elements, /*needsUniqueCopy*/ true, /*multiLine*/(node.flags & NodeFlags.MultiLine) !== 0,
                         /*trailingComma*/ elements.hasTrailingComma);
                 }
             }
@@ -1850,7 +1850,7 @@ var __param = (this && this.__param) || function (paramIndex, decorator) {
                     write("void 0");
                 }
                 write(", ");
-                emitListWithSpread(node.arguments, /*alwaysCopy*/ false, /*multiLine*/ false, /*trailingComma*/ false);
+                emitListWithSpread(node.arguments, /*needsUniqueCopy*/ false, /*multiLine*/ false, /*trailingComma*/ false);
                 write(")");
             }
 
@@ -1913,7 +1913,7 @@ var __param = (this && this.__param) || function (paramIndex, decorator) {
                     write(".bind.apply(");
                     emit(target);
                     write(", [void 0].concat(");
-                    emitListWithSpread(node.arguments, /*alwaysCopy*/true, /*multiline*/false, /*trailingComma*/false);
+                    emitListWithSpread(node.arguments, /*needsUniqueCopy*/false, /*multiline*/false, /*trailingComma*/false);
                     write(")))");
                     write("()");
                 }

Original file line number	Diff line number	Diff line change
`@@ -1366,7 +1366,7 @@ var __param = (this && this.__param) \|\| function (paramIndex, decorator) {`
`1366`	`1366`	`return true;`
`1367`	`1367`	`}`
`1368`	`1368`
`1369`		`- function emitListWithSpread(elements: Expression[], alwaysCopy: boolean, multiLine: boolean, trailingComma: boolean) {`
	`1369`	`+ function emitListWithSpread(elements: Expression[], needsUniqueCopy: boolean, multiLine: boolean, trailingComma: boolean) {`
`1370`	`1370`	`let pos = 0;`
`1371`	`1371`	`let group = 0;`
`1372`	`1372`	`let length = elements.length;`
`@@ -1383,7 +1383,7 @@ var __param = (this && this.__param) \|\| function (paramIndex, decorator) {`
`1383`	`1383`	`e = (<SpreadElementExpression>e).expression;`
`1384`	`1384`	`emitParenthesizedIf(e, /parenthesized/ group === 0 && needsParenthesisForPropertyAccessOrInvocation(e));`
`1385`	`1385`	`pos++;`
`1386`		`- if (pos === length && group === 0 && alwaysCopy && e.kind !== SyntaxKind.ArrayLiteralExpression) {`
	`1386`	`+ if (pos === length && group === 0 && needsUniqueCopy && e.kind !== SyntaxKind.ArrayLiteralExpression) {`
`1387`	`1387`	`write(".slice()");`
`1388`	`1388`	`}`
`1389`	`1389`	`}`
`@@ -1425,7 +1425,7 @@ var __param = (this && this.__param) \|\| function (paramIndex, decorator) {`
`1425`	`1425`	`write("]");`
`1426`	`1426`	`}`
`1427`	`1427`	`else {`
`1428`		`- emitListWithSpread(elements, /alwaysCopy/ true, /multiLine/(node.flags & NodeFlags.MultiLine) !== 0,`
	`1428`	`+ emitListWithSpread(elements, /needsUniqueCopy/ true, /multiLine/(node.flags & NodeFlags.MultiLine) !== 0,`
`1429`	`1429`	`/trailingComma/ elements.hasTrailingComma);`
`1430`	`1430`	`}`
`1431`	`1431`	`}`
`@@ -1850,7 +1850,7 @@ var __param = (this && this.__param) \|\| function (paramIndex, decorator) {`
`1850`	`1850`	`write("void 0");`
`1851`	`1851`	`}`
`1852`	`1852`	`write(", ");`
`1853`		`- emitListWithSpread(node.arguments, /alwaysCopy/ false, /multiLine/ false, /trailingComma/ false);`
	`1853`	`+ emitListWithSpread(node.arguments, /needsUniqueCopy/ false, /multiLine/ false, /trailingComma/ false);`
`1854`	`1854`	`write(")");`
`1855`	`1855`	`}`
`1856`	`1856`
`@@ -1913,7 +1913,7 @@ var __param = (this && this.__param) \|\| function (paramIndex, decorator) {`
`1913`	`1913`	`write(".bind.apply(");`
`1914`	`1914`	`emit(target);`
`1915`	`1915`	`write(", [void 0].concat(");`
`1916`		`- emitListWithSpread(node.arguments, /alwaysCopy/true, /multiline/false, /trailingComma/false);`
	`1916`	`+ emitListWithSpread(node.arguments, /needsUniqueCopy/false, /multiline/false, /trailingComma/false);`
`1917`	`1917`	`write(")))");`
`1918`	`1918`	`write("()");`
`1919`	`1919`	`}`