Skip to content

Commit df425d0

Browse files
authored
Fixed npm run check and linting errors (#13)
1 parent de2874a commit df425d0

File tree

8 files changed

+26
-22
lines changed

8 files changed

+26
-22
lines changed

cli/asc.js

-1
Original file line numberDiff line numberDiff line change
@@ -655,7 +655,6 @@ exports.main = function main(argv, options, callback) {
655655
stats.validateCount++;
656656
stats.validateTime += measure(() => {
657657
if (!module.validate()) {
658-
console.log(module.toText())
659658
module.dispose();
660659
return callback(Error("Validate error"));
661660
}

index.d.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
/// <reference path="./dist/assemblyscript.d.ts" />
2-
export * from "assemblyscript";
1+
import "./src/glue/js";
2+
export * from "./src";

index.js

+7-1
Original file line numberDiff line numberDiff line change
@@ -1 +1,7 @@
1-
module.exports = require("./dist/assemblyscript");
1+
try { require("source-map-support").install(); } catch (e) { /* nop */ }
2+
require("ts-node").register({
3+
project: require("path").join(__dirname, "src", "tsconfig.json"),
4+
skipIgnore: true
5+
});
6+
require("./src/glue/js");
7+
module.exports = require("./src");

package.json

+1
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,7 @@
8282
"check:config": "tsc --noEmit -p src --diagnostics --listFiles",
8383
"check:require": "tsc --noEmit --target ESNEXT --module commonjs --experimentalDecorators tests/require/index",
8484
"check:lint": "eslint --max-warnings 0 --ext js . && eslint --max-warnings 0 --ext ts .",
85+
"check:lint:fix": "eslint --max-warnings 0 --ext js . && eslint --max-warnings 0 --ext ts . --fix",
8586
"test": "npm run test:parser && npm run test:compiler && npm run test:packages && npm run test:extension",
8687
"test:parser": "node tests/parser",
8788
"test:compiler": "node tests/compiler",

src/compiler.ts

-2
Original file line numberDiff line numberDiff line change
@@ -195,8 +195,6 @@ import {
195195
isPowerOf2
196196
} from "./util";
197197

198-
const CLOSURE_TAG = 1 << 31;
199-
200198
/** Compiler options. */
201199
export class Options {
202200

tests/compiler/closure-common-js-patterns.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ let myInstanceThing = ((): IIFEReturn => {
3232
myFunc: (x: i32) => {
3333
return x + 1;
3434
}
35-
}
35+
};
3636
})();
3737

3838
assert(myInstanceThing.myBool == true);
@@ -46,10 +46,10 @@ type generatedFunc = () => i32;
4646
let myFactory = (x: i32): generatedFunc => {
4747
let myFunc = (): i32 => {
4848
return 24 + x;
49-
}
49+
};
5050

5151
return myFunc;
52-
}
52+
};
5353

5454
let generatedPlusOne: generatedFunc = myFactory(1);
5555
let generatedPlusTwo: generatedFunc = myFactory(2);

tests/compiler/closure-passing-functions.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
function add(x: i32, y: i32): i32 {
2-
return x + y
2+
return x + y;
33
}
44
function apply_to_pair(x: i32, y: i32, fn: (x:i32, y:i32) => i32): i32 {
55
return fn(x,y);

tests/compiler/closure.ts

+12-12
Original file line numberDiff line numberDiff line change
@@ -66,18 +66,18 @@ returnOverBoundary();
6666
// KNOWN BUGS
6767

6868
// causes a memory leak, copyFunction is properly released
69-
//const func = (i: i32): i32 => i;
70-
//let copyFunction: (i: i32) => i32 = func;
69+
// const func = (i: i32): i32 => i;
70+
// let copyFunction: (i: i32) => i32 = func;
7171

7272
// also causes a memory leak
73-
//function nestedExecutionTest(arg: i32): i32 {
74-
//var x = 7;
75-
//var f = complexCreateClosure(arg);
76-
//var g = (fn: (x3: i32) => i32): i32 => {
77-
//var first = fn(arg);
78-
//return x;
79-
//};
80-
//return g(f);
81-
//}
82-
//nestedExecutionTest(1);
73+
// function nestedExecutionTest(arg: i32): i32 {
74+
// var x = 7;
75+
// var f = complexCreateClosure(arg);
76+
// var g = (fn: (x3: i32) => i32): i32 => {
77+
// var first = fn(arg);
78+
// return x;
79+
// };
80+
// return g(f);
81+
// }
82+
// nestedExecutionTest(1);
8383

0 commit comments

Comments
 (0)