fix

nodejs · Aug 26, 2024 · 22dd1e0 · 22dd1e0
1 parent 7d3007b
commit 22dd1e0
Showing 1 changed file with 37 additions and 9 deletions.
diff --git a/nan_callbacks_12_inl.h b/nan_callbacks_12_inl.h
@@ -9,9 +9,31 @@
 #ifndef NAN_CALLBACKS_12_INL_H_
 #define NAN_CALLBACKS_12_INL_H_
 
+#if defined(V8_MAJOR_VERSION) &&                                               \
+    (V8_MAJOR_VERSION > 12 ||                                                  \
+     (V8_MAJOR_VERSION == 12 && defined(V8_MINOR_VERSION) &&                   \
+      (V8_MINOR_VERSION > 6 ||                                                 \
+       V8_MINOR_VERSION == 6 && V8_BUILD_NUMBER >= 74)))
+template<typename T>
+class ReturnValue;
+
+namespace imp {
+template<typename T>
+v8::Local<T> imp::GetReturnValue(const ReturnValue<T> &);
+}
+#endif
+
 template<typename T>
 class ReturnValue {
   v8::ReturnValue<T> value_;
+#if defined(V8_MAJOR_VERSION) &&                                               \
+    (V8_MAJOR_VERSION > 12 ||                                                  \
+     (V8_MAJOR_VERSION == 12 && defined(V8_MINOR_VERSION) &&                   \
+      (V8_MINOR_VERSION > 6 ||                                                 \
+       V8_MINOR_VERSION == 6 && V8_BUILD_NUMBER >= 74)))
+  template<typename S>
+  friend v8::Local<S> imp::GetReturnValue(const ReturnValue<S> &rv);
+#endif
 
  public:
   template <class S>
@@ -262,6 +284,12 @@ typedef void (*NativeSetter)(
      (V8_MAJOR_VERSION == 12 && defined(V8_MINOR_VERSION) &&                   \
       (V8_MINOR_VERSION > 6 ||                                                 \
        V8_MINOR_VERSION == 6 && V8_BUILD_NUMBER >= 74)))
+
+template<typename T>
+inline v8::Local<T> GetReturnValue(const ReturnValue<T> &rv) {
+  return rv.value_.Get();
+}
+
 static
 v8::Intercepted PropertyGetterCallbackWrapper(
     v8::Local<v8::Name> property
@@ -274,7 +302,7 @@ v8::Intercepted PropertyGetterCallbackWrapper(
           obj->GetInternalField(kPropertyGetterIndex)
               .As<v8::Value>().As<v8::External>()->Value()));
   callback(property.As<v8::String>(), cbinfo);
-  return cbinfo.GetReturnValue().Get() ==
+  return imp::GetReturnValue(cbinfo.GetReturnValue()) ==
                  v8::Undefined(v8::Isolate::GetCurrent())
              ? v8::Intercepted::kNo
              : v8::Intercepted::kYes;
@@ -319,7 +347,7 @@ v8::Intercepted PropertySetterCallbackWrapper(
           obj->GetInternalField(kPropertySetterIndex)
               .As<v8::Value>().As<v8::External>()->Value()));
   callback(property.As<v8::String>(), value, cbinfo);
-  return cbinfo.GetReturnValue().Get() ==
+  return imp::GetReturnValue(cbinfo.GetReturnValue()) ==
                  v8::Undefined(v8::Isolate::GetCurrent())
              ? v8::Intercepted::kNo
              : v8::Intercepted::kYes;
@@ -384,13 +412,13 @@ v8::Intercepted PropertyDeleterCallbackWrapper(
           obj->GetInternalField(kPropertyDeleterIndex)
               .As<v8::Value>().As<v8::External>()->Value()));
   callback(property.As<v8::String>(), cbinfo);
-  return cbinfo.GetReturnValue().Get() ==
+  return imp::GetReturnValue(cbinfo.GetReturnValue()) ==
                  v8::Undefined(v8::Isolate::GetCurrent())
              ? v8::Intercepted::kNo
              : v8::Intercepted::kYes;
 }
 
-typedef v8::Intercepted (NativePropertyDeleter)
+typedef v8::Intercepted (*NativePropertyDeleter)
     (v8::Local<v8::Name>, const v8::PropertyCallbackInfo<v8::Boolean> &);
 #else
 static
@@ -428,7 +456,7 @@ v8::Intercepted PropertyQueryCallbackWrapper(
           obj->GetInternalField(kPropertyQueryIndex)
               .As<v8::Value>().As<v8::External>()->Value()));
   callback(property.As<v8::String>(), cbinfo);
-  return cbinfo.GetReturnValue().Get() ==
+  return imp::GetReturnValue(cbinfo.GetReturnValue()) ==
                  v8::Undefined(v8::Isolate::GetCurrent())
              ? v8::Intercepted::kNo
              : v8::Intercepted::kYes;
@@ -559,7 +587,7 @@ v8::Intercepted IndexGetterCallbackWrapper(
           obj->GetInternalField(kIndexPropertyGetterIndex)
               .As<v8::Value>().As<v8::External>()->Value()));
   callback(index, cbinfo);
-  return cbinfo.GetReturnValue().Get() ==
+  return imp::GetReturnValue(cbinfo.GetReturnValue()) ==
                  v8::Undefined(v8::Isolate::GetCurrent())
              ? v8::Intercepted::kNo
              : v8::Intercepted::kYes;
@@ -603,7 +631,7 @@ v8::Intercepted IndexSetterCallbackWrapper(
           obj->GetInternalField(kIndexPropertySetterIndex)
               .As<v8::Value>().As<v8::External>()->Value()));
   callback(index, value, cbinfo);
-  return cbinfo.GetReturnValue().Get() ==
+  return imp::GetReturnValue(cbinfo.GetReturnValue()) ==
                  v8::Undefined(v8::Isolate::GetCurrent())
              ? v8::Intercepted::kNo
              : v8::Intercepted::kYes;
@@ -668,7 +696,7 @@ v8::Intercepted IndexDeleterCallbackWrapper(
           obj->GetInternalField(kIndexPropertyDeleterIndex)
               .As<v8::Value>().As<v8::External>()->Value()));
   callback(index, cbinfo);
-  return cbinfo.GetReturnValue().Get() ==
+  return imp::GetReturnValue(cbinfo.GetReturnValue()) ==
                  v8::Undefined(v8::Isolate::GetCurrent())
              ? v8::Intercepted::kNo
              : v8::Intercepted::kYes;
@@ -710,7 +738,7 @@ v8::Intercepted IndexQueryCallbackWrapper(
           obj->GetInternalField(kIndexPropertyQueryIndex)
               .As<v8::Value>().As<v8::External>()->Value()));
   callback(index, cbinfo);
-  return cbinfo.GetReturnValue().Get() ==
+  return imp::GetReturnValue(cbinfo.GetReturnValue()) ==
                  v8::Undefined(v8::Isolate::GetCurrent())
              ? v8::Intercepted::kNo
              : v8::Intercepted::kYes;