net: some scattered cleanup
This commit cleans up net module, including:
- Remove assigning
handle.readable
andhandle.writable
. It's add in https://github.com/nodejs/node-v0.x-archive/pull/3422 and I see nowhere we use these two properties right now. - Documents the enviroment variable of
NODE_PENDING_PIPE_INSTANCES
. It was add in https://github.com/nodejs/node/commit/99c9d19184fdd46ae88464d7a608aac57a530634 long time ago but was not documented and there is no test for it. Maybe we can consider removing it? - Use constants for '0.0.0.0' and '::'.
Checklist
-
make -j4 test
(UNIX), orvcbuild test
(Windows) passes -
tests and/or benchmarks are included -
documentation is changed or added -
commit message follows commit guidelines