diff options
author | Rich Trott <rtrott@gmail.com> | 2016-10-15 15:02:43 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2016-10-19 22:20:27 -0700 |
commit | 68ba9aa0fb6693a1fb5dd114a3ddbe447517c0dd (patch) | |
tree | 9aa5e49fe6db669b58078dbbfcb2231b93e3b93d /test/parallel/test-http-client-readable.js | |
parent | 4b65a65e75f48ff447cabd5500ce115fb5ad4c57 (diff) | |
download | android-node-v8-68ba9aa0fb6693a1fb5dd114a3ddbe447517c0dd.tar.gz android-node-v8-68ba9aa0fb6693a1fb5dd114a3ddbe447517c0dd.tar.bz2 android-node-v8-68ba9aa0fb6693a1fb5dd114a3ddbe447517c0dd.zip |
test,lib,benchmark: match function names
In most cases, named functions match the variable or property to which
they are being assigned. That also seems to be the practice in a series
of PRs currently being evaluated that name currently-anonymous
functions.
This change applies that rule to instances in the code base that don't
comply with that practice.
This will be enforceable with a lint rule once we upgrade to ESLint
3.8.0.
PR-URL: https://github.com/nodejs/node/pull/9113
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Diffstat (limited to 'test/parallel/test-http-client-readable.js')
-rw-r--r-- | test/parallel/test-http-client-readable.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/parallel/test-http-client-readable.js b/test/parallel/test-http-client-readable.js index c035132eb0..3c50dc74f1 100644 --- a/test/parallel/test-http-client-readable.js +++ b/test/parallel/test-http-client-readable.js @@ -15,7 +15,7 @@ FakeAgent.prototype.createConnection = function createConnection() { var s = new Duplex(); var once = false; - s._read = function read() { + s._read = function _read() { if (once) return this.push(null); once = true; @@ -27,7 +27,7 @@ FakeAgent.prototype.createConnection = function createConnection() { }; // Blackhole - s._write = function write(data, enc, cb) { + s._write = function _write(data, enc, cb) { cb(); }; |