Skip to content

lib: replace _linklist with ES6 Sets #2973

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.

Sign up for GitHub

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

Closed
Trott wants to merge 11 commits into from
Closed

lib: replace _linklist with ES6 Sets #2973

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
eb66e7c
lib: replace _linklist with ES6 Set
Trott Sep 21, 2015
b98eb65
fixup: whitespace reinstated
Trott Sep 21, 2015
62a05d1
fixup: remove assert
Trott Sep 22, 2015
7bf7f71
fixup: rm util/debug
Trott Sep 22, 2015
4e10014
fixup: comment update
Trott Sep 23, 2015
e1ccbad
fixup: remove unnecessary delete
Trott Sep 23, 2015
9f28c3c
fixup: keep trigger timers in their own map
Trott Sep 23, 2015
d07e3a7
fixup: rm unnecessary conditional checks
Trott Sep 24, 2015
4510314
fixup: multiple sets -> single ordered array
Trott Sep 26, 2015
1b23f3c
fixup: comments
Trott Sep 26, 2015
bf5c004
fixup: replace all sets with arrays
Trott Sep 26, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixup: remove assert
  • Loading branch information
@Trott
Trott committed Sep 26, 2015
commit 62a05d1bdeb1530314bdcfef1c73ec7171664088
2 changes: 0 additions & 2 deletions lib/timers.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
'use strict';

const Timer = process.binding('timer_wrap').Timer;
const assert = require('assert').ok;
const util = require('util');
const debug = util.debuglog('timer');
const kOnTimeout = Timer.kOnTimeout | 0;
Expand Down Expand Up @@ -587,7 +586,6 @@ function unrefTimeout() {
exports._unrefActive = function(item) {
var msecs = item._idleTimeout;
if (!msecs || msecs < 0) return;
assert(msecs >= 0);

if (lists[msecs]) {
lists[msecs].delete(item);
Expand Down