Skip to content

Commit

Permalink
src: reduce variable scope in node_trace_writer.cc
Browse files Browse the repository at this point in the history
PR-URL: #23297
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gabriel Schulhof <gabriel.schulhof@intel.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
cjihrig authored and targos committed Oct 10, 2018
1 parent 046fd98 commit 56c2f57
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/tracing/node_trace_writer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,8 @@ void NodeTraceWriter::WriteSuffix() {
NodeTraceWriter::~NodeTraceWriter() {
WriteSuffix();
uv_fs_t req;
int err;
if (fd_ != -1) {
err = uv_fs_close(nullptr, &req, fd_, nullptr);
CHECK_EQ(err, 0);
CHECK_EQ(0, uv_fs_close(nullptr, &req, fd_, nullptr));
uv_fs_req_cleanup(&req);
}
uv_async_send(&exit_signal_);
Expand Down

0 comments on commit 56c2f57

Please sign in to comment.