Remove options argument from LuaTransformer

ark120202 · ark120202 · commit f2ffcccaee41 · 2019-04-22T07:05:23.000+05:00
diff --git a/src/LuaTransformer.ts b/src/LuaTransformer.ts
@@ -43,12 +43,11 @@ export class LuaTransformer {
         "not", "or", "repeat", "return", "self", "then", "until", "while",
     ]);
 
-    private isStrict = true;
+    private isStrict: boolean;
     private luaTarget: LuaTarget;
 
     private checker: ts.TypeChecker;
     protected options: CompilerOptions;
-    protected program: ts.Program;
 
     private isModule = false;
 
@@ -69,17 +68,16 @@ export class LuaTransformer {
 
     private readonly typeValidationCache: Map<ts.Type, Set<ts.Type>> = new Map<ts.Type, Set<ts.Type>>();
 
-    public constructor(program: ts.Program, options: CompilerOptions) {
+    public constructor(protected program: ts.Program) {
         this.checker = program.getTypeChecker();
-        this.options = options;
-        this.program = program;
+        this.options = program.getCompilerOptions();
         this.isStrict = this.options.alwaysStrict !== undefined
                         || (this.options.strict !== undefined && this.options.alwaysStrict !== false)
                         || (this.isModule
                             && this.options.target !== undefined
                             && this.options.target >= ts.ScriptTarget.ES2015);
 
-        this.luaTarget = options.luaTarget || LuaTarget.LuaJIT;
+        this.luaTarget = this.options.luaTarget || LuaTarget.LuaJIT;
 
         this.setupState();
     }
diff --git a/test/util.ts b/test/util.ts
@@ -93,11 +93,8 @@ export function executeLua(luaStr: string, withLib = true): any {
 }
 
 // Get a mock transformer to use for testing
-export function makeTestTransformer(
-    target: tstl.LuaTarget = tstl.LuaTarget.Lua53,
-): tstl.LuaTransformer {
-    const options = { luaTarget: target };
-    return new tstl.LuaTransformer(ts.createProgram([], options), options);
+export function makeTestTransformer(luaTarget = tstl.LuaTarget.Lua53): tstl.LuaTransformer {
+    return new tstl.LuaTransformer(ts.createProgram([], { luaTarget }));
 }
 
 export function transpileAndExecute(

Original file line number	Diff line number	Diff line change
`@@ -93,11 +93,8 @@ export function executeLua(luaStr: string, withLib = true): any {`
`93`	`93`	`}`
`94`	`94`
`95`	`95`	`// Get a mock transformer to use for testing`
`96`		`-export function makeTestTransformer(`
`97`		`- target: tstl.LuaTarget = tstl.LuaTarget.Lua53,`
`98`		`-): tstl.LuaTransformer {`
`99`		`- const options = { luaTarget: target };`
`100`		`- return new tstl.LuaTransformer(ts.createProgram([], options), options);`
	`96`	`+export function makeTestTransformer(luaTarget = tstl.LuaTarget.Lua53): tstl.LuaTransformer {`
	`97`	`+ return new tstl.LuaTransformer(ts.createProgram([], { luaTarget }));`
`101`	`98`	`}`
`102`	`99`
`103`	`100`	`export function transpileAndExecute(`