Home Original page

libs: fix some RegExp nits by vsemozhetbyt · Pull Request #13536 · nodejs/node

added 3 commits

June 8, 2017 02:24

@nodejs-github-bot nodejs-github-bot added dns

Issues and PRs related to the dns subsystem.

repl

Issues and PRs related to the REPL subsystem.

tls

Issues and PRs related to the tls subsystem.

util

Issues and PRs related to the built-in util module.

labels

Jun 7, 2017

addaleax

vsemozhetbyt added a commit that referenced this pull request

Jun 10, 2017 
* Take RegExp creation out of cycles.
* Use test(), not match() in boolean context.
* Remove redundant RegExp parts.

PR-URL: #13536
Reviewed-By: Anna Henningsen <anna@addaleax.net>

addaleax pushed a commit that referenced this pull request

Jun 10, 2017 
* Take RegExp creation out of cycles.
* Use test(), not match() in boolean context.
* Remove redundant RegExp parts.

PR-URL: #13536
Reviewed-By: Anna Henningsen <anna@addaleax.net>

MylesBorins pushed a commit that referenced this pull request

Jul 18, 2017 
* Take RegExp creation out of cycles.
* Use test(), not match() in boolean context.
* Remove redundant RegExp parts.

Backport-PR-URL: #14348
PR-URL: #13536
Reviewed-By: Anna Henningsen <anna@addaleax.net>

MylesBorins pushed a commit that referenced this pull request

Jul 21, 2017 
* Take RegExp creation out of cycles.
* Use test(), not match() in boolean context.
* Remove redundant RegExp parts.

Backport-PR-URL: #14348
PR-URL: #13536
Reviewed-By: Anna Henningsen <anna@addaleax.net>