From d224816ed1095f5daf844f689d683c61c6f1a3a8 Mon Sep 17 00:00:00 2001 From: David Vacca Date: Sun, 1 Oct 2023 18:02:16 -0700 Subject: [PATCH] Reduce visibility of JSInstance (#39738) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/39738 In an attempt to reduce footprint of React Native Android public APIs we are reducing visibility of classes and interfaces that are not meant to be used publicly OR are public but have no usages. As part of our analysis, which involved looking for usages inside the Meta codebase and code search in OSS, we've detected that this class/interface is public but it's not used from other packages. If you are using this class or interface please comment in this PR and we will restate the public access. changelog: [Android][Changed] Reducing visibility of JSInstance Reviewed By: RSNara Differential Revision: D49752139 fbshipit-source-id: b56667a91abd16d15a321dfc3e83026b596eaad3 --- .../src/main/java/com/facebook/react/bridge/JSInstance.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/JSInstance.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/JSInstance.java index 3b5a5abf14a765..e4eef1716eaa8f 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/JSInstance.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/JSInstance.java @@ -11,7 +11,7 @@ * This interface includes the methods needed to use a running JS instance, without specifying any * of the bridge-specific initialization or lifecycle management. */ -public interface JSInstance { +interface JSInstance { void invokeCallback(int callbackID, NativeArrayInterface arguments); // TODO if this interface survives refactoring, think about adding // callFunction.