fixup! fs: add c++ fast path for writeFileSync utf8

Author: CanadaHonk

src/node_file.cc

@@ -2489,6 +2489,8 @@ static void WriteFileUtf8(const FunctionCallbackInfo<Value>& args) {
 
   CHECK_GE(args.Length(), 4);
 
+  CHECK(args[1]->IsString());
+
   CHECK(args[2]->IsInt32());
   const int flags = args[2].As<Int32>()->Value();
 
@@ -2511,31 +2513,28 @@ static void WriteFileUtf8(const FunctionCallbackInfo<Value>& args) {
     FS_SYNC_TRACE_BEGIN(open);
     file = uv_fs_open(nullptr, &req, *path, flags, mode, nullptr);
     FS_SYNC_TRACE_END(open);
+
     if (req.result < 0) {
       uv_fs_req_cleanup(&req);
       // req will be cleaned up by scope leave.
       return env->ThrowUVException(req.result, "open", nullptr, path.out());
     }
   }
 
-  auto defer_close = OnScopeLeave([file, is_fd, &req]() {
-    if (!is_fd) {
-      FS_SYNC_TRACE_BEGIN(close);
-      CHECK_EQ(0, uv_fs_close(nullptr, &req, file, nullptr));
-      FS_SYNC_TRACE_END(close);
-    }
-    uv_fs_req_cleanup(&req);
-  });
-
-  CHECK(args[1]->IsString());
   node::Utf8Value value(isolate, args[1]);
-
   uv_buf_t uvbuf = uv_buf_init(*value, value.length());
 
   FS_SYNC_TRACE_BEGIN(write);
   int err = uv_fs_write(nullptr, &req, file, &uvbuf, 1, 0, nullptr);
   FS_SYNC_TRACE_END(write);
 
+  if (!is_fd) {
+    FS_SYNC_TRACE_BEGIN(close);
+    CHECK_EQ(0, uv_fs_close(nullptr, &req, file, nullptr));
+    FS_SYNC_TRACE_END(close);
+  }
+  uv_fs_req_cleanup(&req);
+
   if (err < 0) {
     return env->ThrowUVException(err, "write");
   }