New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Timer, feat: Compatible with Nodejs' setTimeout function, the parameter timeout can be undefined. #446
Conversation
test/timer_test.js
Outdated
@@ -228,20 +244,20 @@ describe("timer", () => { | |||
}); | |||
}); | |||
|
|||
describe("clearInterval in callback", () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
提交时过滤不必要的格式化
另外 fibjs 内置了 vsc 格式化配置,所以尽量使用 vsc,以避免日后代码修改时的反复格式化
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已经处理(shift+option+f)
test/timer_test.js
Outdated
|
||
GC(); | ||
var no5 = test_util.countObject('Timer'); | ||
assert.equal(no1 + 1, no5); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
缺省参数可以单独一个用例,不要和已有的用例混在一起。尽量不要修改已有用例,除非存在不兼容的情况。
好像漏提交了 idl 文件 |
idl 中英文两个文件也要提交 |
No description provided.