Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions src/LuaLib.ts
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ export enum LuaLibFeature {
InstanceOf = "InstanceOf",
InstanceOfObject = "InstanceOfObject",
Iterator = "Iterator",
LuaIteratorSpread = "LuaIteratorSpread",
Map = "Map",
MathAtan2 = "MathAtan2",
MathSign = "MathSign",
Expand Down
13 changes: 13 additions & 0 deletions src/lualib/LuaIteratorSpread.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
export function __TS__LuaIteratorSpread<TKey, TValue, TState>(
this: (this: void, state: TState, key: TKey) => LuaMultiReturn<[TKey, TValue]>,
state: TState,
firstKey: TKey
): LuaMultiReturn<Array<[TKey, TValue]>> {
const results = [];
let [key, value] = this(state, firstKey);
while (key) {
results.push([key, value]);
[key, value] = this(state, key);
}
return $multi(...results);
}
4 changes: 4 additions & 0 deletions src/transformation/utils/language-extensions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,10 @@ export enum IterableExtensionKind {
PairsKey = "PairsKey",
}

export function isLuaIterable(context: TransformationContext, type: ts.Type): boolean {
return getPropertyValue(context, type, "__tstlIterable") !== undefined;
}

export function getIterableExtensionTypeForType(
context: TransformationContext,
type: ts.Type
Expand Down
5 changes: 5 additions & 0 deletions src/transformation/visitors/spread.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import * as ts from "typescript";
import * as lua from "../../LuaAST";
import { FunctionVisitor, TransformationContext } from "../context";
import { isLuaIterable } from "../utils/language-extensions";
import { createUnpackCall } from "../utils/lua-ast";
import { LuaLibFeature, transformLuaLibFunction } from "../utils/lualib";
import {
Expand Down Expand Up @@ -73,6 +74,10 @@ export const transformSpreadElement: FunctionVisitor<ts.SpreadElement> = (node,
if (isMultiReturnCall(context, tsInnerExpression)) return innerExpression;

const type = context.checker.getTypeAtLocation(node.expression); // not ts-inner expression, in case of casts
if (isLuaIterable(context, type)) {
return transformLuaLibFunction(context, LuaLibFeature.LuaIteratorSpread, node, innerExpression);
}

if (isArrayType(context, type)) {
return createUnpackCall(context, innerExpression, node);
}
Expand Down
40 changes: 40 additions & 0 deletions test/unit/spread.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -478,3 +478,43 @@ describe("vararg spread in IIFE", () => {
`.expectToMatchJsResult();
});
});

// https://github.com/TypeScriptToLua/TypeScriptToLua/issues/1244
test.each(["pairs", "ipairs"])("can spread %s (#1244)", func => {
util.testFunction`
const arr = ["a", "b", "c"];
return [...${func}(arr)];
`
.withLanguageExtensions()
.setTsHeader(
`
declare function ipairs<T>(this: void, t: T): LuaIterable<LuaMultiReturn<[number, NonNullable<T[keyof T]>]>>;
declare function pairs<T>(this: void, t: T): LuaIterable<LuaMultiReturn<[keyof T, NonNullable<T[keyof T]>]>>;
`
)
.expectToEqual([
[1, "a"],
[2, "b"],
[3, "c"],
]);
});

// https://github.com/TypeScriptToLua/TypeScriptToLua/issues/1244
test.each(["LuaTable", "LuaMap"])("can spread %s (#1244)", type => {
const result: Array<[string, string]> = util.testFunction`
const tbl = new ${type}();
tbl.set("foo", "bar");
tbl.set("fizz", "buzz");
return [...pairs(tbl)];
`
.withLanguageExtensions()
.setTsHeader(
"declare function pairs<T>(this: void, t: T): LuaIterable<LuaMultiReturn<[keyof T, NonNullable<T[keyof T]>]>>;"
)
.getLuaExecutionResult();

// We don't know the order so match like this
expect(result).toHaveLength(2);
expect(result.some(([k, v]) => k === "foo" && v === "bar")).toBe(true);
expect(result.some(([k, v]) => k === "fizz" && v === "buzz")).toBe(true);
});