Home Original page

timers: remove unused repeat param in timer_wrap by jscissr · Pull Request #7994 · nodejs/node

@jscissr 

The `repeat` param in `start(timeout, repeat)` was 0 in all callsites.

@nodejs-github-bot added c++

Issues and PRs that require attention from people who are familiar with C++.

timers

Issues and PRs related to the timers subsystem / setImmediate, setInterval, setTimeout.

labels

Aug 6, 2016

@mscdex mscdex removed the c++

Issues and PRs that require attention from people who are familiar with C++.

label

Aug 6, 2016

jasnell pushed a commit that referenced this pull request

Aug 9, 2016 
The `repeat` param in `start(timeout, repeat)` was 0 in all callsites.

PR-URL: #7994
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

cjihrig pushed a commit that referenced this pull request

Aug 10, 2016 
The `repeat` param in `start(timeout, repeat)` was 0 in all callsites.

PR-URL: #7994
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>