From 3c0ad81fef96d23a7240b708346c7f015f37aea9 Mon Sep 17 00:00:00 2001 From: TatianaKapos Date: Fri, 14 Apr 2023 13:07:09 -0700 Subject: [PATCH] Comment out unreferenced formal parameter (#36853) Summary: Recent windows integration brought in a change where a unreferenced formal parameter was introduced in TurboModule.h. Windows treats with warning as an error, this change comments out the formal parameter name to get rid of the error/warning. ## Changelog: [GENERAL] [FIXED] - Comment out unreferenced formal parameter Pull Request resolved: https://github.com/facebook/react-native/pull/36853 Test Plan: Passed Windows testcases Reviewed By: javache Differential Revision: D45007332 Pulled By: philIip fbshipit-source-id: 2e8128549d628a8be6bcd76c1fa473d0bc27649b --- .../react/nativemodule/core/ReactCommon/TurboModule.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModule.h b/packages/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModule.h index e6a3109fb7d5dc..ecc7e39ba284e7 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModule.h +++ b/packages/react-native/ReactCommon/react/nativemodule/core/ReactCommon/TurboModule.h @@ -120,7 +120,7 @@ class JSI_EXPORT TurboModule : public facebook::jsi::HostObject { static_cast(meta.argCount), [this, meta]( jsi::Runtime &rt, - const jsi::Value &thisVal, + [[maybe_unused]] const jsi::Value &thisVal, const jsi::Value *args, size_t count) { return meta.invoker(rt, *this, args, count); }); }