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

12483 Commits

131 Branches

446 Tags

325 MiB

Tree: 4118fd5794

Commit Graph

Author SHA1 Message Date

Author	SHA1	Message	Date
Jonas Dohse	e2b8393ee8	test: port domains regression test from v0.10 `f2a45caf2e` contained a test for a regression that had been introduced by the original change that `77a10ed05f` ported. While `77a10ed05f` did not contain that regression, the test that `f2a45caf2e` contained should still be in the code base to prevent any regression from happening in the future. Original message for the commit that contained the test: domains: fix stack clearing after error handled `caeb67735b` introduced a regression where the domains stack would not be cleared after an error had been handled by the top-level domain. This change clears the domains stack regardless of the position of the active domain in the stack. PR: #9364 PR-URL: https://github.com/joyent/node/pull/9364 Reviewed-By: Trevor Norris <trev.norris@gmail.com> Reviewed-By: Julien Gilli <julien.gilli@joyent.com> PR: #3356 PR-URL: https://github.com/nodejs/node/pull/3356 Reviewed-By: Ben Noordhuis <ben@strongloop.com>	10 years ago

Jonas Dohse

e2b8393ee8

test: port domains regression test from v0.10

f2a45caf2e contained a test for a
regression that had been introduced by the original change that
77a10ed05f ported. While
77a10ed05f did not contain that
regression, the test that f2a45caf2e
contained should still be in the code base to prevent any regression
from happening in the future.

Original message for the commit that contained the test:

  domains: fix stack clearing after error handled

  caeb67735b introduced a regression where
  the domains stack would not be cleared after an error had been handled
  by the top-level domain.

  This change clears the domains stack regardless of the position of the
  active domain in the stack.

  PR: #9364
  PR-URL: https://github.com/joyent/node/pull/9364
  Reviewed-By: Trevor Norris <trev.norris@gmail.com>
  Reviewed-By: Julien Gilli <julien.gilli@joyent.com>

PR: #3356
PR-URL: https://github.com/nodejs/node/pull/3356
Reviewed-By: Ben Noordhuis <ben@strongloop.com>

10 years ago

1 Commits (4118fd579499c18906ab75e9b0cea76044458a37)