Made LuaJIT use table.unpack for destructing assignments (#257)

Perryvw · web-flow · commit abb9f07e94f6 · 2018-10-19T22:27:56.000+02:00
diff --git a/src/targets/Transpiler.JIT.ts b/src/targets/Transpiler.JIT.ts
@@ -34,4 +34,9 @@ export class LuaTranspilerJIT extends LuaTranspiler52 {
                 throw TSTLErrors.UnsupportedKind("bitwise operator", node.operatorToken.kind, node);
         }
     }
+
+    /** @override */
+    public transpileDestructingAssignmentValue(node: ts.Expression): string {
+        return `table.unpack(${this.transpileExpression(node)})`;
+    }
 }
diff --git a/test/unit/assignmentDestructuring.spec.ts b/test/unit/assignmentDestructuring.spec.ts
@@ -9,7 +9,7 @@ export class AssignmentDestructuringTests {
         let [a, b] = myFunc();`;
 
     @Test("Assignment destructuring [5.1]")
-    public assignmentDestructuring51() {
+    public assignmentDestructuring51(): void {
         // Transpile
         const lua = util.transpileString(
             this.assignmentDestruturingTs, {luaTarget: LuaTarget.Lua51, luaLibImport: "none"}
@@ -19,12 +19,22 @@ export class AssignmentDestructuringTests {
     }
 
     @Test("Assignment destructuring [5.2]")
-    public tupleDestructing52() {
+    public tupleDestructing52(): void {
         // Transpile
         const lua = util.transpileString(
             this.assignmentDestruturingTs, {luaTarget: LuaTarget.Lua52, luaLibImport: "none"}
         );
         // Assert
         Expect(lua).toBe(`local a,b=table.unpack(myFunc());`);
     }
+
+    @Test("Assignment destructuring [JIT]")
+    public assignmentDestructuringJIT(): void {
+        // Transpile
+        const lua = util.transpileString(
+            this.assignmentDestruturingTs, {luaTarget: LuaTarget.LuaJIT, luaLibImport: "none"}
+        );
+        // Assert
+        Expect(lua).toBe(`local a,b=table.unpack(myFunc());`);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -34,4 +34,9 @@ export class LuaTranspilerJIT extends LuaTranspiler52 {`
`34`	`34`	`throw TSTLErrors.UnsupportedKind("bitwise operator", node.operatorToken.kind, node);`
`35`	`35`	`}`
`36`	`36`	`}`
	`37`	`+`
	`38`	`+ /** @override */`
	`39`	`+ public transpileDestructingAssignmentValue(node: ts.Expression): string {`
	`40`	+ return `table.unpack(${this.transpileExpression(node)})`;
	`41`	`+ }`
`37`	`42`	`}`