address PR feedback

src/js_native_api_v8.cc

@@ -604,10 +604,6 @@ void Finalizer::ResetFinalizer() {
   finalize_hint_ = nullptr;
 }
 
-void Finalizer::ResetData(void* data) {
-  finalize_data_ = data;
-}
-
 void Finalizer::CallFinalizer() {
   napi_finalize finalize_callback = finalize_callback_;
   void* finalize_data = finalize_data_;

src/js_native_api_v8.h

@@ -8,7 +8,7 @@ inline napi_status napi_clear_last_error(node_api_nogc_env env);
 
 namespace v8impl {
 
-// Base class to track references and finalizers in a double linked list.
+// Base class to track references and finalizers in a doubly linked list.
 class RefTracker {
  public:
   using RefList = RefTracker;
@@ -334,7 +334,6 @@ class Finalizer {
 
   void ResetEnv();
   void ResetFinalizer();
-  void ResetData(void* data);
   void CallFinalizer();
 
  private:

src/node_api.cc

@@ -121,17 +121,16 @@ namespace {
 class BufferFinalizer : private Finalizer {
  public:
   static BufferFinalizer* New(napi_env env,
-                              napi_finalize finalize_callback = nullptr,
-                              void* finalize_data = nullptr,
-                              void* finalize_hint = nullptr) {
+                              napi_finalize finalize_callback,
+                              void* finalize_data,
+                              void* finalize_hint) {
     return new BufferFinalizer(
         env, finalize_callback, finalize_data, finalize_hint);
   }
   // node::Buffer::FreeCallback
   static void FinalizeBufferCallback(char* data, void* hint) {
     std::unique_ptr<BufferFinalizer, Deleter> finalizer{
         static_cast<BufferFinalizer*>(hint)};
-    finalizer->ResetData(data);
     // It is safe to call into JavaScript at this point.
     finalizer->CallFinalizer();
   }
@@ -1053,7 +1052,7 @@ napi_create_external_buffer(napi_env env,
 
   // The finalizer object will delete itself after invoking the callback.
   v8impl::BufferFinalizer* finalizer =
-      v8impl::BufferFinalizer::New(env, finalize_cb, nullptr, finalize_hint);
+      v8impl::BufferFinalizer::New(env, finalize_cb, data, finalize_hint);
 
   v8::MaybeLocal<v8::Object> maybe =
       node::Buffer::New(isolate,