From 1bfd92a321f19e92f00c48c6f6f13a8d34a8a780 Mon Sep 17 00:00:00 2001 From: Nick Gerleman Date: Thu, 4 Jan 2024 01:43:38 -0800 Subject: [PATCH] Remove suppressions for Wgnu-zero-variadic-macro-arguments (#42136) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/42136 `Wpedantic` flags usage of variadic macros with zero arguments. This is widely supported by different compilers (including MSVC), but was previously forbidden by the standard. C++ 20 explicitly allows them, so, theoretically Clang should know not to warn about these now. Let's try that. Changelog: [Internal] Differential Revision: D52534129 fbshipit-source-id: 1a951fef553ea4dc54ed71d44374da408dcc1f0a --- .../ReactAndroid/src/main/jni/react/jni/JCallback.h | 7 ------- packages/react-native/ReactCommon/React-Fabric.podspec | 2 +- .../react-native/ReactCommon/React-FabricImage.podspec | 2 +- packages/react-native/ReactCommon/ReactCommon.podspec | 2 +- packages/react-native/ReactCommon/butter/CMakeLists.txt | 3 +-- .../react-native/ReactCommon/callinvoker/CMakeLists.txt | 3 +-- .../jserrorhandler/React-jserrorhandler.podspec | 2 +- .../react-native/ReactCommon/react/bridging/CMakeLists.txt | 1 - .../react-native/ReactCommon/react/config/CMakeLists.txt | 1 - .../react-native/ReactCommon/react/debug/CMakeLists.txt | 1 - .../ReactCommon/react/nativemodule/core/CMakeLists.txt | 1 - .../core/platform/ios/React-NativeModulesApple.podspec | 2 +- .../react/nativemodule/samples/ReactCommon-Samples.podspec | 2 +- .../nativemodule/samples/platform/android/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/animations/CMakeLists.txt | 1 - .../react/renderer/attributedstring/CMakeLists.txt | 1 - .../react/renderer/componentregistry/CMakeLists.txt | 1 - .../react/renderer/componentregistry/native/CMakeLists.txt | 1 - .../react/renderer/components/image/CMakeLists.txt | 1 - .../components/legacyviewmanagerinterop/CMakeLists.txt | 1 - .../react/renderer/components/modal/CMakeLists.txt | 1 - .../react/renderer/components/progressbar/CMakeLists.txt | 1 - .../react/renderer/components/root/CMakeLists.txt | 1 - .../react/renderer/components/scrollview/CMakeLists.txt | 1 - .../react/renderer/components/switch/CMakeLists.txt | 1 - .../react/renderer/components/text/CMakeLists.txt | 1 - .../react/renderer/components/textinput/CMakeLists.txt | 1 - .../renderer/components/unimplementedview/CMakeLists.txt | 1 - .../react/renderer/components/view/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/core/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/debug/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/element/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/graphics/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/imagemanager/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/leakchecker/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/mapbuffer/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/mounting/CMakeLists.txt | 1 - .../react/renderer/runtimescheduler/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/scheduler/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/telemetry/CMakeLists.txt | 1 - .../react/renderer/textlayoutmanager/CMakeLists.txt | 1 - .../ReactCommon/react/renderer/uimanager/CMakeLists.txt | 1 - .../ReactCommon/react/runtime/React-RuntimeCore.podspec | 2 +- .../ReactCommon/react/runtime/React-RuntimeHermes.podspec | 2 +- .../react/runtime/platform/ios/React-RuntimeApple.podspec | 2 +- .../react-native/ReactCommon/react/utils/CMakeLists.txt | 1 - .../ReactCommon/reactperflogger/CMakeLists.txt | 3 +-- .../ReactCommon/runtimeexecutor/CMakeLists.txt | 3 +-- packages/rn-tester/NativeCxxModuleExample/CMakeLists.txt | 1 - 49 files changed, 13 insertions(+), 59 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/jni/react/jni/JCallback.h b/packages/react-native/ReactAndroid/src/main/jni/react/jni/JCallback.h index 807f95cd8cf7a4..0220f800d3d3fe 100644 --- a/packages/react-native/ReactAndroid/src/main/jni/react/jni/JCallback.h +++ b/packages/react-native/ReactAndroid/src/main/jni/react/jni/JCallback.h @@ -29,16 +29,9 @@ class JCxxCallbackImpl : public jni::HybridClass { "Lcom/facebook/react/bridge/CxxCallbackImpl;"; static void registerNatives() { -#if __clang__ -#pragma clang diagnostic push -#pragma clang diagnostic ignored "-Wgnu-zero-variadic-macro-arguments" -#endif javaClassStatic()->registerNatives({ makeNativeMethod("nativeInvoke", JCxxCallbackImpl::invoke), }); -#if __clang__ -#pragma clang diagnostic pop -#endif } private: diff --git a/packages/react-native/ReactCommon/React-Fabric.podspec b/packages/react-native/ReactCommon/React-Fabric.podspec index c9ee015fddcd13..eb89c2c43e07ca 100644 --- a/packages/react-native/ReactCommon/React-Fabric.podspec +++ b/packages/react-native/ReactCommon/React-Fabric.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' folly_dep_name = 'RCT-Folly/Fabric' boost_compiler_flags = '-Wno-documentation' diff --git a/packages/react-native/ReactCommon/React-FabricImage.podspec b/packages/react-native/ReactCommon/React-FabricImage.podspec index a32f07c8b9207a..3230e852aad12d 100644 --- a/packages/react-native/ReactCommon/React-FabricImage.podspec +++ b/packages/react-native/ReactCommon/React-FabricImage.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' folly_dep_name = 'RCT-Folly/Fabric' boost_compiler_flags = '-Wno-documentation' diff --git a/packages/react-native/ReactCommon/ReactCommon.podspec b/packages/react-native/ReactCommon/ReactCommon.podspec index 0d7fa415fb1222..55479772afe102 100644 --- a/packages/react-native/ReactCommon/ReactCommon.podspec +++ b/packages/react-native/ReactCommon/ReactCommon.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' boost_compiler_flags = '-Wno-documentation' using_hermes = ENV['USE_HERMES'] == nil || ENV['USE_HERMES'] == "1" diff --git a/packages/react-native/ReactCommon/butter/CMakeLists.txt b/packages/react-native/ReactCommon/butter/CMakeLists.txt index 98e65cc1f0a79e..1bfbc89275e996 100644 --- a/packages/react-native/ReactCommon/butter/CMakeLists.txt +++ b/packages/react-native/ReactCommon/butter/CMakeLists.txt @@ -12,8 +12,7 @@ add_compile_options( -frtti -std=c++20 -Wall - -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments) + -Wpedantic) add_library(butter INTERFACE) diff --git a/packages/react-native/ReactCommon/callinvoker/CMakeLists.txt b/packages/react-native/ReactCommon/callinvoker/CMakeLists.txt index ec5b8172cae158..e2abeb2dbb9742 100644 --- a/packages/react-native/ReactCommon/callinvoker/CMakeLists.txt +++ b/packages/react-native/ReactCommon/callinvoker/CMakeLists.txt @@ -11,8 +11,7 @@ add_compile_options( -frtti -std=c++20 -Wall - -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments) + -Wpedantic) add_library(callinvoker INTERFACE) diff --git a/packages/react-native/ReactCommon/jserrorhandler/React-jserrorhandler.podspec b/packages/react-native/ReactCommon/jserrorhandler/React-jserrorhandler.podspec index 28f5ddc3dd9bf5..266793e20ff946 100644 --- a/packages/react-native/ReactCommon/jserrorhandler/React-jserrorhandler.podspec +++ b/packages/react-native/ReactCommon/jserrorhandler/React-jserrorhandler.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' folly_dep_name = 'RCT-Folly/Fabric' boost_compiler_flags = '-Wno-documentation' diff --git a/packages/react-native/ReactCommon/react/bridging/CMakeLists.txt b/packages/react-native/ReactCommon/react/bridging/CMakeLists.txt index f084beccab6294..666441ef19aa15 100644 --- a/packages/react-native/ReactCommon/react/bridging/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/bridging/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"ReactNative\") file(GLOB react_bridging_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/config/CMakeLists.txt b/packages/react-native/ReactCommon/react/config/CMakeLists.txt index 6885b6510f432a..387d983876f513 100644 --- a/packages/react-native/ReactCommon/react/config/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/config/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_config_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/debug/CMakeLists.txt b/packages/react-native/ReactCommon/react/debug/CMakeLists.txt index ec0ec12da81283..4b811df4c31b55 100644 --- a/packages/react-native/ReactCommon/react/debug/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/debug/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") diff --git a/packages/react-native/ReactCommon/react/nativemodule/core/CMakeLists.txt b/packages/react-native/ReactCommon/react/nativemodule/core/CMakeLists.txt index 40888fcbde83f8..bdfb55d18ab8a4 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/core/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/nativemodule/core/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"ReactNative\") diff --git a/packages/react-native/ReactCommon/react/nativemodule/core/platform/ios/React-NativeModulesApple.podspec b/packages/react-native/ReactCommon/react/nativemodule/core/platform/ios/React-NativeModulesApple.podspec index fbb82c5fe30786..2ab4ef46279b43 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/core/platform/ios/React-NativeModulesApple.podspec +++ b/packages/react-native/ReactCommon/react/nativemodule/core/platform/ios/React-NativeModulesApple.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' boost_compiler_flags = '-Wno-documentation' using_hermes = ENV['USE_HERMES'] == nil || ENV['USE_HERMES'] == "1" diff --git a/packages/react-native/ReactCommon/react/nativemodule/samples/ReactCommon-Samples.podspec b/packages/react-native/ReactCommon/react/nativemodule/samples/ReactCommon-Samples.podspec index 0e07d5817b7a58..8a6516cd332f6c 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/samples/ReactCommon-Samples.podspec +++ b/packages/react-native/ReactCommon/react/nativemodule/samples/ReactCommon-Samples.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' boost_compiler_flags = '-Wno-documentation' using_hermes = ENV['USE_HERMES'] == nil || ENV['USE_HERMES'] == "1" diff --git a/packages/react-native/ReactCommon/react/nativemodule/samples/platform/android/CMakeLists.txt b/packages/react-native/ReactCommon/react/nativemodule/samples/platform/android/CMakeLists.txt index 2e032601b0044d..b405fc7c78836a 100644 --- a/packages/react-native/ReactCommon/react/nativemodule/samples/platform/android/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/nativemodule/samples/platform/android/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DFOLLY_NO_CONFIG=1 -DLOG_TAG=\"ReactNative\") diff --git a/packages/react-native/ReactCommon/react/renderer/animations/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/animations/CMakeLists.txt index 200d2f590db58d..a8d115021a926c 100644 --- a/packages/react-native/ReactCommon/react/renderer/animations/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/animations/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_animations_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/attributedstring/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/attributedstring/CMakeLists.txt index 0215258ebccab8..fbdedd5849fff2 100644 --- a/packages/react-native/ReactCommon/react/renderer/attributedstring/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/attributedstring/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_attributedstring_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/componentregistry/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/componentregistry/CMakeLists.txt index 7cc08f89db1c09..ed5cb2e737f8e6 100644 --- a/packages/react-native/ReactCommon/react/renderer/componentregistry/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/componentregistry/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_componentregistry_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/componentregistry/native/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/componentregistry/native/CMakeLists.txt index 2fecbecf153905..f5a38f9707c962 100644 --- a/packages/react-native/ReactCommon/react/renderer/componentregistry/native/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/componentregistry/native/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_native_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/image/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/image/CMakeLists.txt index 9dbf4cbe43395e..9dbe78d706dfa0 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/image/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/image/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_image_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/legacyviewmanagerinterop/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/legacyviewmanagerinterop/CMakeLists.txt index e340c9d8dc7f9b..f2419281270b6f 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/legacyviewmanagerinterop/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/legacyviewmanagerinterop/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_legacyviewmanagerinterop_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/modal/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/modal/CMakeLists.txt index 5dd94549149fd5..2fa4021b39b703 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/modal/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/modal/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_modal_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/progressbar/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/progressbar/CMakeLists.txt index c72594888e5a37..663b608bff4cc2 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/progressbar/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/progressbar/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_progressbar_SRC CONFIGURE_DEPENDS android/react/renderer/components/progressbar/*.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/root/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/root/CMakeLists.txt index 7c94f04ae0fdc8..6aa6e77cce7870 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/root/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/root/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_root_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/scrollview/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/scrollview/CMakeLists.txt index 6276b415491a16..4e58b4d1c90c7c 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/scrollview/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/scrollview/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_scrollview_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/switch/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/switch/CMakeLists.txt index acc0fa27426c1e..422d3afe4954c0 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/switch/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/switch/CMakeLists.txt @@ -42,5 +42,4 @@ target_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments ) diff --git a/packages/react-native/ReactCommon/react/renderer/components/text/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/text/CMakeLists.txt index 7ecf8389397abe..394d72d7de529b 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/text/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/text/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_text_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/textinput/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/textinput/CMakeLists.txt index 748501abb89d59..e8d12d59df91fd 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/textinput/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/textinput/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_textinput_SRC CONFIGURE_DEPENDS platform/android/react/renderer/components/androidtextinput/*.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/unimplementedview/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/unimplementedview/CMakeLists.txt index 1b023e66255da5..1f6996eebfb482 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/unimplementedview/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/unimplementedview/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_unimplementedview_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/components/view/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/components/view/CMakeLists.txt index a6ad1aeea58083..2cf299bcbb4bcc 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/view/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/components/view/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB rrc_view_SRC CONFIGURE_DEPENDS diff --git a/packages/react-native/ReactCommon/react/renderer/core/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/core/CMakeLists.txt index 8dc03e6ef55828..7e24d29f97bb45 100644 --- a/packages/react-native/ReactCommon/react/renderer/core/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/core/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_core_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/debug/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/debug/CMakeLists.txt index d5271e9987892b..4b41809f5a79c0 100644 --- a/packages/react-native/ReactCommon/react/renderer/debug/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/debug/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_debug_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/element/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/element/CMakeLists.txt index 362871fb92185b..d7c4b8dd39bb19 100644 --- a/packages/react-native/ReactCommon/react/renderer/element/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/element/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_element_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/graphics/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/graphics/CMakeLists.txt index 05914513e8af4a..ab3977ceb5711a 100644 --- a/packages/react-native/ReactCommon/react/renderer/graphics/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/graphics/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_graphics_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/imagemanager/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/imagemanager/CMakeLists.txt index 17c3040c65f21e..8a69855f9ac321 100644 --- a/packages/react-native/ReactCommon/react/renderer/imagemanager/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/imagemanager/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_imagemanager_SRC CONFIGURE_DEPENDS diff --git a/packages/react-native/ReactCommon/react/renderer/leakchecker/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/leakchecker/CMakeLists.txt index e78e664b991282..4f9dde26baaa6e 100644 --- a/packages/react-native/ReactCommon/react/renderer/leakchecker/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/leakchecker/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_leakchecker_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/mapbuffer/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/mapbuffer/CMakeLists.txt index 1962c29acc6b0d..95bf23caa1598b 100644 --- a/packages/react-native/ReactCommon/react/renderer/mapbuffer/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/mapbuffer/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_mapbuffer_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/mounting/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/mounting/CMakeLists.txt index 8a7e0fab4efacc..cc96aa3731dd6e 100644 --- a/packages/react-native/ReactCommon/react/renderer/mounting/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/mounting/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_mounting_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/runtimescheduler/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/runtimescheduler/CMakeLists.txt index e88e1a96d413a6..b8ad2984c8e563 100644 --- a/packages/react-native/ReactCommon/react/renderer/runtimescheduler/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/runtimescheduler/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_runtimescheduler_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/scheduler/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/scheduler/CMakeLists.txt index cef146cf5d1373..38fdcb66e8639c 100644 --- a/packages/react-native/ReactCommon/react/renderer/scheduler/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/scheduler/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_scheduler_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/telemetry/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/telemetry/CMakeLists.txt index 202f66b11be9bf..a0fa3f8b948de2 100644 --- a/packages/react-native/ReactCommon/react/renderer/telemetry/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/telemetry/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_telemetry_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/renderer/textlayoutmanager/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/textlayoutmanager/CMakeLists.txt index 0946144d91b8e7..4333c91e2b988a 100644 --- a/packages/react-native/ReactCommon/react/renderer/textlayoutmanager/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/textlayoutmanager/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_textlayourmanager_SRC CONFIGURE_DEPENDS diff --git a/packages/react-native/ReactCommon/react/renderer/uimanager/CMakeLists.txt b/packages/react-native/ReactCommon/react/renderer/uimanager/CMakeLists.txt index acd92e019384ff..a7583450cadfb7 100644 --- a/packages/react-native/ReactCommon/react/renderer/uimanager/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/renderer/uimanager/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_render_uimanager_SRC CONFIGURE_DEPENDS *.cpp) diff --git a/packages/react-native/ReactCommon/react/runtime/React-RuntimeCore.podspec b/packages/react-native/ReactCommon/react/runtime/React-RuntimeCore.podspec index eeb146b73ce9ab..ec8d9219b11cac 100644 --- a/packages/react-native/ReactCommon/react/runtime/React-RuntimeCore.podspec +++ b/packages/react-native/ReactCommon/react/runtime/React-RuntimeCore.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' folly_dep_name = 'RCT-Folly/Fabric' boost_compiler_flags = '-Wno-documentation' diff --git a/packages/react-native/ReactCommon/react/runtime/React-RuntimeHermes.podspec b/packages/react-native/ReactCommon/react/runtime/React-RuntimeHermes.podspec index b8bd53b8aac458..7bb462409b98cb 100644 --- a/packages/react-native/ReactCommon/react/runtime/React-RuntimeHermes.podspec +++ b/packages/react-native/ReactCommon/react/runtime/React-RuntimeHermes.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' folly_dep_name = 'RCT-Folly/Fabric' boost_compiler_flags = '-Wno-documentation' diff --git a/packages/react-native/ReactCommon/react/runtime/platform/ios/React-RuntimeApple.podspec b/packages/react-native/ReactCommon/react/runtime/platform/ios/React-RuntimeApple.podspec index a56449fbaac581..9087518d7b714f 100644 --- a/packages/react-native/ReactCommon/react/runtime/platform/ios/React-RuntimeApple.podspec +++ b/packages/react-native/ReactCommon/react/runtime/platform/ios/React-RuntimeApple.podspec @@ -16,7 +16,7 @@ else source[:tag] = "v#{version}" end -folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32 -Wno-gnu-zero-variadic-macro-arguments' +folly_compiler_flags = '-DFOLLY_NO_CONFIG -DFOLLY_MOBILE=1 -DFOLLY_USE_LIBCPP=1 -DFOLLY_CFG_NO_COROUTINES=1 -DFOLLY_HAVE_CLOCK_GETTIME=1 -Wno-comma -Wno-shorten-64-to-32' folly_version = '2023.08.07.00' folly_dep_name = 'RCT-Folly/Fabric' boost_compiler_flags = '-Wno-documentation' diff --git a/packages/react-native/ReactCommon/react/utils/CMakeLists.txt b/packages/react-native/ReactCommon/react/utils/CMakeLists.txt index 495648afa6b365..9328122311c5dc 100644 --- a/packages/react-native/ReactCommon/react/utils/CMakeLists.txt +++ b/packages/react-native/ReactCommon/react/utils/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DLOG_TAG=\"Fabric\") file(GLOB react_utils_SRC CONFIGURE_DEPENDS *.cpp *.mm) diff --git a/packages/react-native/ReactCommon/reactperflogger/CMakeLists.txt b/packages/react-native/ReactCommon/reactperflogger/CMakeLists.txt index d3120f62a8e76f..1f19cda575dfda 100644 --- a/packages/react-native/ReactCommon/reactperflogger/CMakeLists.txt +++ b/packages/react-native/ReactCommon/reactperflogger/CMakeLists.txt @@ -11,8 +11,7 @@ add_compile_options( -frtti -std=c++20 -Wall - -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments) + -Wpedantic) file(GLOB reactperflogger_SRC CONFIGURE_DEPENDS reactperflogger/*.cpp) add_library(reactperflogger STATIC ${reactperflogger_SRC}) diff --git a/packages/react-native/ReactCommon/runtimeexecutor/CMakeLists.txt b/packages/react-native/ReactCommon/runtimeexecutor/CMakeLists.txt index 315341a91fe698..2ff283f8bcb74f 100644 --- a/packages/react-native/ReactCommon/runtimeexecutor/CMakeLists.txt +++ b/packages/react-native/ReactCommon/runtimeexecutor/CMakeLists.txt @@ -11,8 +11,7 @@ add_compile_options( -frtti -std=c++20 -Wall - -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments) + -Wpedantic) file(GLOB_RECURSE runtimeexecutor_SRC CONFIGURE_DEPENDS *.cpp *.h) diff --git a/packages/rn-tester/NativeCxxModuleExample/CMakeLists.txt b/packages/rn-tester/NativeCxxModuleExample/CMakeLists.txt index 0fe23d107d9e16..e017b80962aa8a 100644 --- a/packages/rn-tester/NativeCxxModuleExample/CMakeLists.txt +++ b/packages/rn-tester/NativeCxxModuleExample/CMakeLists.txt @@ -12,7 +12,6 @@ add_compile_options( -std=c++20 -Wall -Wpedantic - -Wno-gnu-zero-variadic-macro-arguments -DFOLLY_NO_CONFIG=1 -DLOG_TAG=\"ReactNative\")