net: refactor onSlaveClose in Server.close (!12334) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/claudiorodriguez/net-close-arrow into master Apr 11, 2017

Refactors onSlaveClose in Server.close to be an arrow function, removes need for self = this and moves it down to make code more readable.

make bench-net before and after (don't see any significant change): https://gist.github.com/claudiorodriguez/b3b9483c05bd6c21626c9bcd0bfa7897

Checklist

make -j4 test (UNIX), or vcbuild test (Windows) passes
commit message follows commit guidelines

Affected core subsystem(s)

net