Skip to content

Commit ca1bae6

Browse files
sashashakunaddaleax
authored andcommitted
test: remove unused args and comparison fix
Remove unused arguments and change non-strict comparison to the strict one in test. PR-URL: #11396 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
1 parent 66f9506 commit ca1bae6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/internet/test-dgram-multicast-multi-process.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -22,14 +22,14 @@ if (common.inFreeBSDJail) {
2222
return;
2323
}
2424

25-
function launchChildProcess(index) {
25+
function launchChildProcess() {
2626
const worker = fork(__filename, ['child']);
2727
workers[worker.pid] = worker;
2828

2929
worker.messagesReceived = [];
3030

3131
// Handle the death of workers.
32-
worker.on('exit', function(code, signal) {
32+
worker.on('exit', function(code) {
3333
// Don't consider this the true death if the worker has finished
3434
// successfully or if the exit code is 0.
3535
if (worker.isDone || code === 0) {
@@ -189,7 +189,7 @@ if (process.argv[2] === 'child') {
189189

190190
process.send({ message: buf.toString() });
191191

192-
if (receivedMessages.length == messages.length) {
192+
if (receivedMessages.length === messages.length) {
193193
// .dropMembership() not strictly needed but here as a sanity check.
194194
listenSocket.dropMembership(LOCAL_BROADCAST_HOST);
195195
process.nextTick(function() {

0 commit comments

Comments
 (0)