From d7d4989bdbc438e8ee12a76f2cf206bc665b5bf9 Mon Sep 17 00:00:00 2001 From: David Vacca Date: Mon, 23 Oct 2023 20:02:38 -0700 Subject: [PATCH] Update javadoc on BaseJavaModule (#41103) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/41103 Update javadoc on BaseJavaModule to remove references to bridge classes bypass-github-export-checks changelog: [internal] internal Reviewed By: luluwu2032 Differential Revision: D50294832 fbshipit-source-id: bca7be9e2e6a9ebee3e040b4b7e0d5c320627e26 --- .../com/facebook/react/bridge/BaseJavaModule.java | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/BaseJavaModule.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/BaseJavaModule.java index 99bd57571e6..780cf404b44 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/BaseJavaModule.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/bridge/BaseJavaModule.java @@ -92,7 +92,10 @@ public abstract class BaseJavaModule implements NativeModule { onCatalystInstanceDestroy(); } - /** Subclasses can use this method to access catalyst context passed as a constructor. */ + /** + * Subclasses can use this method to access {@link ReactApplicationContext} passed as a + * constructor. + */ protected final ReactApplicationContext getReactApplicationContext() { return Assertions.assertNotNull( mReactApplicationContext, @@ -100,10 +103,11 @@ public abstract class BaseJavaModule implements NativeModule { } /** - * Subclasses can use this method to access catalyst context passed as a constructor. Use this - * version to check that the underlying CatalystInstance is active before returning, and - * automatically have SoftExceptions or debug information logged for you. Consider using this - * whenever calling ReactApplicationContext methods that require the Catalyst instance be alive. + * Subclasses can use this method to access {@link ReactApplicationContext} passed as a + * constructor. Use this version to check that the underlying React Instance is active before + * returning, and automatically have SoftExceptions or debug information logged for you. Consider + * using this whenever calling ReactApplicationContext methods that require the React instance be + * alive. * *

This can return null at any time, but especially during teardown methods it's * possible/likely.