diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2017-10-25 10:02:54 +0200 |
|---|---|---|
| committer | Anna Henningsen <anna@addaleax.net> | 2017-10-29 11:39:40 +0100 |
| commit | 7c3d6ccbf2101580d7704f8c37de09ef109ba841 (patch) | |
| tree | d7447257c27947e81b3a5db0e7032bf93a033f69 /src/udp_wrap.cc | |
| parent | a4e46df87fc92e9c0da7a011486e40a34ef13307 (diff) | |
| download | android-node-v8-7c3d6ccbf2101580d7704f8c37de09ef109ba841.tar.gz android-node-v8-7c3d6ccbf2101580d7704f8c37de09ef109ba841.tar.bz2 android-node-v8-7c3d6ccbf2101580d7704f8c37de09ef109ba841.zip | |
src: remove superfluous HandleScope
Accessors implicitly run inside a HandleScope, UDPWrap::GetFD() doesn't
need to create one explicitly.
PR-URL: https://github.com/nodejs/node/pull/16482
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Diffstat (limited to 'src/udp_wrap.cc')
| -rw-r--r-- | src/udp_wrap.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/udp_wrap.cc b/src/udp_wrap.cc index 98399155a8..4a37aadc66 100644 --- a/src/udp_wrap.cc +++ b/src/udp_wrap.cc @@ -169,7 +169,6 @@ void UDPWrap::New(const FunctionCallbackInfo<Value>& args) { void UDPWrap::GetFD(Local<String>, const PropertyCallbackInfo<Value>& args) { int fd = UV_EBADF; #if !defined(_WIN32) - HandleScope scope(args.GetIsolate()); UDPWrap* wrap = Unwrap<UDPWrap>(args.Holder()); if (wrap != nullptr) uv_fileno(reinterpret_cast<uv_handle_t*>(&wrap->handle_), &fd); |