From fdae17daf966e9a6cfeee6254dad7b633fdbf38b Mon Sep 17 00:00:00 2001 From: David Vacca Date: Thu, 14 Sep 2023 11:13:16 -0700 Subject: [PATCH] Fix lint warning (#39338) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/39338 Fix lint warning in MainReactPackage, as reactModule could be null changelog: [internal] internal Reviewed By: cortinico Differential Revision: D49077064 fbshipit-source-id: 34e04cac1a8875af94bbb78119a323c6319b6d6b --- .../react/shell/MainReactPackage.java | 22 ++++++++++--------- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/shell/MainReactPackage.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/shell/MainReactPackage.java index da406d8bedb..a686584f6ee 100644 --- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/shell/MainReactPackage.java +++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/shell/MainReactPackage.java @@ -284,16 +284,18 @@ public class MainReactPackage extends TurboReactPackage implements ViewManagerOn for (Class moduleClass : moduleList) { ReactModule reactModule = moduleClass.getAnnotation(ReactModule.class); - reactModuleInfoMap.put( - reactModule.name(), - new ReactModuleInfo( - reactModule.name(), - moduleClass.getName(), - reactModule.canOverrideExistingModule(), - reactModule.needsEagerInit(), - reactModule.hasConstants(), - reactModule.isCxxModule(), - TurboModule.class.isAssignableFrom(moduleClass))); + if (reactModule != null) { + reactModuleInfoMap.put( + reactModule.name(), + new ReactModuleInfo( + reactModule.name(), + moduleClass.getName(), + reactModule.canOverrideExistingModule(), + reactModule.needsEagerInit(), + reactModule.hasConstants(), + reactModule.isCxxModule(), + TurboModule.class.isAssignableFrom(moduleClass))); + } } return () -> reactModuleInfoMap;