Use map for usedProperties

ark120202 · ark120202 · commit e2349f0fea0f · 2019-05-22T02:01:43.000+05:00
diff --git a/src/LuaTransformer.ts b/src/LuaTransformer.ts
@@ -1390,8 +1390,10 @@ export class LuaTransformer {
                     const elements = pattern.elements as ts.NodeArray<ts.BindingElement>;
                     const usedProperties = elements.map(e =>
                         tstl.createTableFieldExpression(
+                            tstl.createBooleanLiteral(true),
                             tstl.createStringLiteral(
-                                this.getIdentifierText((e.propertyName || e.name) as ts.Identifier)
+                                this.getIdentifierText((e.propertyName || e.name) as ts.Identifier),
+                                e.propertyName || e.name
                             )
                         )
                     );
diff --git a/src/lualib/ObjectRest.ts b/src/lualib/ObjectRest.ts
@@ -1,19 +1,11 @@
 function __TS__ObjectRest<K extends keyof any, V>(
     this: void,
     target: Record<K, V>,
-    usedProperties: K[]
+    usedProperties: Partial<Record<K, true>>
 ): Partial<Record<K, V>> {
     const result: Partial<Record<K, V>> = {};
     for (const property in target) {
-        let isUsed = false;
-        for (const usedProperty of usedProperties) {
-            if (property === usedProperty) {
-                isUsed = true;
-                break;
-            }
-        }
-
-        if (!isUsed) {
+        if (!usedProperties[property]) {
             result[property] = target[property];
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -1390,8 +1390,10 @@ export class LuaTransformer {`
`1390`	`1390`	`const elements = pattern.elements as ts.NodeArray<ts.BindingElement>;`
`1391`	`1391`	`const usedProperties = elements.map(e =>`
`1392`	`1392`	`tstl.createTableFieldExpression(`
	`1393`	`+ tstl.createBooleanLiteral(true),`
`1393`	`1394`	`tstl.createStringLiteral(`
`1394`		`- this.getIdentifierText((e.propertyName \|\| e.name) as ts.Identifier)`
	`1395`	`+ this.getIdentifierText((e.propertyName \|\| e.name) as ts.Identifier),`
	`1396`	`+ e.propertyName \|\| e.name`
`1395`	`1397`	`)`
`1396`	`1398`	`)`
`1397`	`1399`	`);`