lukechilds/node - node - Gitea: Git with a cup of tea

12352 Commits

131 Branches

446 Tags

325 MiB

Tree: 3eecdf9f14

Commit Graph

Author SHA1 Message Date

Author	SHA1	Message	Date
Ryan Graham	bde32f8a4d	lib: fix undefined timeout regression `63644dd1cd` introduced a regression caused by everyone's favourite JavaScript feature: undefined < 0 === undefined >= 0. Add a case to the existing tests to cover this scenario and then add the check for undefined that makes the test pass. PR-URL: https://github.com/nodejs/node/pull/3331 Reviewed-By: Fedor Indutny <fedor@indutny.com> Reviewed By: Evan Lucas <evanlucas@me.com>	9 years ago
Rich Trott	070aac4a87	lib: remove redundant code, add tests in timers.js insert() is only called from one place where there is already a check that msecs is greater than or equal to zero, so do not repeat the check inside insert(). timers.active() is not documented and should not be exposed, but since it is exposed for now, let's test it. PR-URL: https://github.com/nodejs/node/pull/3143 Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>	9 years ago

Ryan Graham

bde32f8a4d

lib: fix undefined timeout regression

63644dd1cd introduced a regression caused by everyone's favourite
JavaScript feature: undefined < 0 === undefined >= 0.

Add a case to the existing tests to cover this scenario and then add
the check for undefined that makes the test pass.

PR-URL: https://github.com/nodejs/node/pull/3331
Reviewed-By: Fedor Indutny <fedor@indutny.com>
Reviewed By: Evan Lucas <evanlucas@me.com>

9 years ago

Rich Trott

070aac4a87

lib: remove redundant code, add tests in timers.js

insert() is only called from one place where there is already a check
that msecs is greater than or equal to zero, so do not repeat the check
inside insert().

timers.active() is not documented and should not be exposed, but since
it is exposed for now, let's test it.

PR-URL: https://github.com/nodejs/node/pull/3143
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>

9 years ago

2 Commits (3eecdf9f1422640c7439507cb39ffb3dff57b3e4)