From 61ae2012519a4ec842bb13896986753d35043af4 Mon Sep 17 00:00:00 2001 From: Nick Gerleman Date: Mon, 11 Sep 2023 19:51:40 -0700 Subject: [PATCH] Breaking: Remove YGConfigGetInstanceCount Summary: X-link: https://github.com/facebook/yoga/pull/1370 X-link: https://github.com/facebook/react-native/pull/39369 This was added in https://github.com/facebook/yoga/pull/497 specifically for tests related to memory leaks in the C# bindings to count how often YGConfigFree. This is the wrong layer for this check, we don't have officially supported C# bindings anymore, and this API is not safe when Yoga runs on multiple threads. This removes it, similar to a global node instance count that was also previously removed. Reviewed By: rshest Differential Revision: D49131207 fbshipit-source-id: 58537ed635ed455ff065471bdf77061a4bf826f4 --- lib/yoga/src/main/cpp/yoga/Yoga.cpp | 8 -------- lib/yoga/src/main/cpp/yoga/Yoga.h | 1 - 2 files changed, 9 deletions(-) diff --git a/lib/yoga/src/main/cpp/yoga/Yoga.cpp b/lib/yoga/src/main/cpp/yoga/Yoga.cpp index 9a91e032bcb..b9a5961d44d 100644 --- a/lib/yoga/src/main/cpp/yoga/Yoga.cpp +++ b/lib/yoga/src/main/cpp/yoga/Yoga.cpp @@ -172,8 +172,6 @@ YOGA_EXPORT void YGNodeMarkDirtyAndPropagateToDescendants( return resolveRef(node)->markDirtyAndPropagateDownwards(); } -int32_t gConfigInstanceCount = 0; - YOGA_EXPORT WIN_EXPORT YGNodeRef YGNodeNewWithConfig(const YGConfigRef config) { auto* node = new yoga::Node{resolveRef(config)}; yoga::assertFatal( @@ -261,23 +259,17 @@ YOGA_EXPORT void YGNodeReset(YGNodeRef node) { resolveRef(node)->reset(); } -YOGA_EXPORT int32_t YGConfigGetInstanceCount(void) { - return gConfigInstanceCount; -} - YOGA_EXPORT YGConfigRef YGConfigNew(void) { #ifdef ANDROID const YGConfigRef config = new yoga::Config(YGAndroidLog); #else const YGConfigRef config = new yoga::Config(YGDefaultLog); #endif - gConfigInstanceCount++; return config; } YOGA_EXPORT void YGConfigFree(const YGConfigRef config) { delete resolveRef(config); - gConfigInstanceCount--; } YOGA_EXPORT void YGNodeSetIsReferenceBaseline( diff --git a/lib/yoga/src/main/cpp/yoga/Yoga.h b/lib/yoga/src/main/cpp/yoga/Yoga.h index 41464e1ee5b..c9efb85fd8a 100644 --- a/lib/yoga/src/main/cpp/yoga/Yoga.h +++ b/lib/yoga/src/main/cpp/yoga/Yoga.h @@ -330,7 +330,6 @@ void YGConfigSetUseLegacyStretchBehaviour( // YGConfig WIN_EXPORT YGConfigRef YGConfigNew(void); WIN_EXPORT void YGConfigFree(YGConfigRef config); -WIN_EXPORT int32_t YGConfigGetInstanceCount(void); WIN_EXPORT void YGConfigSetExperimentalFeatureEnabled( YGConfigRef config,