From 5f381dbf1097d90507962234c2b767060194c8cd Mon Sep 17 00:00:00 2001 From: David Vacca Date: Sat, 29 Apr 2023 08:52:34 -0700 Subject: [PATCH] Reduce visibility of internals of BridgelessAtomicRef (#37165) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/37165 Reduce visibility of internals of BridgelessAtomicRef changelog: [internal] internal Reviewed By: philIip Differential Revision: D45378245 fbshipit-source-id: c9cd6b7afacc4f0db079cd13cb7c32b44b22cdfe --- .../com/facebook/react/bridgeless/BridgelessAtomicRef.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridgeless/BridgelessAtomicRef.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridgeless/BridgelessAtomicRef.java index 97463ba8f8d..d613b497c9e 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridgeless/BridgelessAtomicRef.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridgeless/BridgelessAtomicRef.java @@ -12,6 +12,7 @@ import static com.facebook.infer.annotation.Assertions.assertNotNull; import android.annotation.SuppressLint; import androidx.annotation.Nullable; import com.facebook.infer.annotation.Nullsafe; +import java.util.Objects; @Nullsafe(Nullsafe.Mode.LOCAL) class BridgelessAtomicRef { @@ -30,8 +31,8 @@ class BridgelessAtomicRef { Failure } - volatile State state; - volatile String failureMessage; + private volatile State state; + private volatile String failureMessage; public BridgelessAtomicRef(@Nullable T initialValue) { mValue = initialValue; @@ -77,7 +78,7 @@ class BridgelessAtomicRef { synchronized (this) { state = State.Failure; String message = ex.getMessage(); - failureMessage = message != null ? message : "null"; + failureMessage = Objects.toString(message, "null"); notifyAll(); }