diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/TextLayoutManager.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/TextLayoutManager.java index d367b0b66bd..14aa27b5138 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/TextLayoutManager.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/TextLayoutManager.java @@ -825,7 +825,7 @@ public class TextLayoutManager { // where the container is measured smaller than text. Math.ceil prevents it // See T136756103 for investigation if (android.os.Build.VERSION.SDK_INT > Build.VERSION_CODES.Q) { - calculatedWidth = Math.min((float) Math.ceil(calculatedWidth), width); + calculatedWidth = (float) Math.ceil(calculatedWidth); } return calculatedWidth; } diff --git a/packages/react-native/ReactCommon/react/renderer/components/view/YogaLayoutableShadowNode.cpp b/packages/react-native/ReactCommon/react/renderer/components/view/YogaLayoutableShadowNode.cpp index a0d9aba0f32..0abf41a8854 100644 --- a/packages/react-native/ReactCommon/react/renderer/components/view/YogaLayoutableShadowNode.cpp +++ b/packages/react-native/ReactCommon/react/renderer/components/view/YogaLayoutableShadowNode.cpp @@ -15,11 +15,9 @@ #include #include #include -#include #include #include #include -#include #include #include #include @@ -839,21 +837,6 @@ YGSize YogaLayoutableShadowNode::yogaNodeMeasureCallbackConnector( auto size = shadowNode.measureContent( threadLocalLayoutContext, {minimumSize, maximumSize}); -#ifdef REACT_NATIVE_DEBUG - bool widthInBounds = size.width + kDefaultEpsilon >= minimumSize.width && - size.width - kDefaultEpsilon <= maximumSize.width; - bool heightInBounds = size.height + kDefaultEpsilon >= minimumSize.height && - size.height - kDefaultEpsilon <= maximumSize.height; - - if (!widthInBounds || !heightInBounds) { - LOG(FATAL) << shadowNode.getComponentDescriptor().getComponentName() - << " returned in invalid measurement. Min: [" - << minimumSize.width << "," << minimumSize.height << "] Max: [" - << maximumSize.width << "," << maximumSize.height - << "] Actual: [" << size.width << "," << size.height << "]"; - } -#endif - return YGSize{ yogaFloatFromFloat(size.width), yogaFloatFromFloat(size.height)}; } diff --git a/packages/react-native/ReactCommon/react/renderer/textlayoutmanager/platform/cxx/react/renderer/textlayoutmanager/TextLayoutManager.cpp b/packages/react-native/ReactCommon/react/renderer/textlayoutmanager/platform/cxx/react/renderer/textlayoutmanager/TextLayoutManager.cpp index cef69312beb..9fa1cec511f 100644 --- a/packages/react-native/ReactCommon/react/renderer/textlayoutmanager/platform/cxx/react/renderer/textlayoutmanager/TextLayoutManager.cpp +++ b/packages/react-native/ReactCommon/react/renderer/textlayoutmanager/platform/cxx/react/renderer/textlayoutmanager/TextLayoutManager.cpp @@ -17,7 +17,7 @@ TextMeasurement TextLayoutManager::measure( const AttributedStringBox& attributedStringBox, const ParagraphAttributes& /*paragraphAttributes*/, const TextLayoutContext& /*layoutContext*/, - const LayoutConstraints& layoutConstraints) const { + const LayoutConstraints& /*layoutConstraints*/) const { TextMeasurement::Attachments attachments; for (const auto& fragment : attributedStringBox.getValue().getFragments()) { if (fragment.isAttachment()) { @@ -25,10 +25,7 @@ TextMeasurement TextLayoutManager::measure( TextMeasurement::Attachment{{{0, 0}, {0, 0}}, false}); } } - return TextMeasurement{ - {layoutConstraints.minimumSize.width, - layoutConstraints.minimumSize.height}, - attachments}; + return TextMeasurement{{0, 0}, attachments}; } } // namespace facebook::react diff --git a/packages/react-native/ReactCommon/react/utils/FloatComparison.h b/packages/react-native/ReactCommon/react/utils/FloatComparison.h index 3e60643a9c3..d760cef1cdf 100644 --- a/packages/react-native/ReactCommon/react/utils/FloatComparison.h +++ b/packages/react-native/ReactCommon/react/utils/FloatComparison.h @@ -9,9 +9,7 @@ namespace facebook::react { -constexpr float kDefaultEpsilon = 0.005f; - -inline bool floatEquality(float a, float b, float epsilon = kDefaultEpsilon) { +inline bool floatEquality(float a, float b, float epsilon = 0.005f) { return (std::isnan(a) && std::isnan(b)) || (!std::isnan(a) && !std::isnan(b) && fabs(a - b) < epsilon); }