diff options
| author | Daniel Bevenius <daniel.bevenius@gmail.com> | 2017-05-18 08:29:43 +0200 |
|---|---|---|
| committer | Daniel Bevenius <daniel.bevenius@gmail.com> | 2017-05-22 07:22:51 +0200 |
| commit | 762b0358b3f5de55a74fe40403964774f80ad33d (patch) | |
| tree | 6ba4be0bfa6d7da7b93b68efb8ade166e7e1411b /src/node_crypto_clienthello.cc | |
| parent | 7a15eefbba113cd0df99b420de1718aa7a01afa8 (diff) | |
| download | node-new-762b0358b3f5de55a74fe40403964774f80ad33d.tar.gz | |
src: remove unnecessary return statement
I think the code would be a little clearer if the return statement
was removed here since End() does not return any value and neither
does ParseHeader.
PR-URL: https://github.com/nodejs/node/pull/13094
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Diffstat (limited to 'src/node_crypto_clienthello.cc')
| -rw-r--r-- | src/node_crypto_clienthello.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/node_crypto_clienthello.cc b/src/node_crypto_clienthello.cc index dc3a692d53..794d877e38 100644 --- a/src/node_crypto_clienthello.cc +++ b/src/node_crypto_clienthello.cc @@ -119,7 +119,7 @@ void ClientHelloParser::ParseHeader(const uint8_t* data, size_t avail) { return; fail: - return End(); + End(); } |