Home Original page

lib: introduce constant MAX_TICKS by danbev · Pull Request #11199 · nodejs/node

@danbev 

Currently the maximum number of tick is duplicated in two places. This
commit introduces a constant that both can use.

jasnell

@addaleax addaleax added the lib / src

Issues and PRs related to general changes in the lib or src directory.

label

Feb 6, 2017

@danbev

@danbev

addaleax

bnoordhuis

trevnorris

@danbev

trevnorris

jasnell pushed a commit that referenced this pull request

Feb 17, 2017 
Currently the maximum number of tick is duplicated in two places. This
commit introduces a constant that both can use.

PR-URL: #11199
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

@danbev danbev deleted the introduce-max-ticks-constant branch

February 17, 2017 06:33

italoacasas pushed a commit to italoacasas/node that referenced this pull request

Feb 20, 2017 
Currently the maximum number of tick is duplicated in two places. This
commit introduces a constant that both can use.

PR-URL: nodejs#11199
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

italoacasas pushed a commit that referenced this pull request

Feb 22, 2017 
Currently the maximum number of tick is duplicated in two places. This
commit introduces a constant that both can use.

PR-URL: #11199
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

jasnell pushed a commit that referenced this pull request

Feb 24, 2017 
PR-URL: #11473
Ref: #11199 (comment)
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

addaleax pushed a commit that referenced this pull request

Feb 24, 2017 
PR-URL: #11473
Ref: #11199 (comment)
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

jasnell pushed a commit that referenced this pull request

Mar 7, 2017 
Currently the maximum number of tick is duplicated in two places. This
commit introduces a constant that both can use.

PR-URL: #11199
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

jasnell pushed a commit that referenced this pull request

Mar 7, 2017 
PR-URL: #11473
Ref: #11199 (comment)
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Mar 9, 2017 
Currently the maximum number of tick is duplicated in two places. This
commit introduces a constant that both can use.

PR-URL: #11199
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trevor Norris <trev.norris@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Mar 9, 2017 
PR-URL: #11473
Ref: #11199 (comment)
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>