★ wanayoo — archive 1999 https://github.com/nodejs/node/commit/ee64ae0f6dNouvelle recherche | Portail wanayoo
Skip to content
Permalink
Browse files
test: remove unused function arguments in async-hooks tests
Remove unused function arguments in two async-hooks tests.

PR-URL: #24368
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
  • Loading branch information
Trott authored and rvagg committed Nov 28, 2018
1 parent 5d67eec commit ee64ae0f6d8c9c4b5800a07d2654595c464d8727
Showing with 2 additions and 2 deletions.
  1. +1 −1 test/async-hooks/test-getaddrinforeqwrap.js
  2. +1 −1 test/async-hooks/test-getnameinforeqwrap.js
@@ -14,7 +14,7 @@ const hooks = initHooks();

hooks.enable();
dns.lookup('www.google.com', 4, common.mustCall(onlookup));
function onlookup(err_, ip, family) {
function onlookup() {
// we don't care about the error here in order to allow
// tests to run offline (lookup will fail in that case and the err be set);

@@ -14,7 +14,7 @@ const hooks = initHooks();

hooks.enable();
dns.lookupService('127.0.0.1', 80, common.mustCall(onlookupService));
function onlookupService(err_, ip, family) {
function onlookupService() {
// we don't care about the error here in order to allow
// tests to run offline (lookup will fail in that case and the err be set)

0 comments on commit ee64ae0

Please sign in to comment.