From bdfac9c04fd5b76b31675e7a70f633d2bbee0c2d Mon Sep 17 00:00:00 2001
From: David Vacca <dvacca@meta.com>
Date: Wed, 11 Feb 2026 14:09:33 -0800
Subject: [PATCH] Fix multiple lint warnings in NativeViewHierarchyManager
 (#55499)

Summary:

Fixed multiple lint warnings in NativeViewHierarchyManager.java:

1. ClownyBooleanExpression: Added SuppressLint for intentionally disabled
   DEBUG_MODE (ReactBuildConfig.DEBUG && false). This pattern is used to
   easily enable debug mode by changing to just ReactBuildConfig.DEBUG.

2. NullableAnnotationShouldNotAddToPrimitiveTypes: Removed Nullable from
   getInstanceHandle() which returns a primitive long. Primitives can't be
   null; this method throws exceptions instead of returning null.

3. NotInvokedPrivateMethod: Removed unused private methods arrayContains()
   and getReactContextForView() which were dead code.

changelog: [internal] internal

Reviewed By: alanleedev

Differential Revision: D91992935
---
 .../uimanager/NativeViewHierarchyManager.java | 29 +++----------------
 1 file changed, 4 insertions(+), 25 deletions(-)

diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/NativeViewHierarchyManager.java b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/NativeViewHierarchyManager.java
index 1ace964c5fb790..65ff5df8981400 100644
--- a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/NativeViewHierarchyManager.java
+++ b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/NativeViewHierarchyManager.java
@@ -7,6 +7,7 @@
 
 package com.facebook.react.uimanager;
 
+import android.annotation.SuppressLint;
 import android.graphics.Matrix;
 import android.graphics.Rect;
 import android.graphics.RectF;
@@ -76,6 +77,9 @@ public class NativeViewHierarchyManager {
   }
 
   private static final String TAG = NativeViewHierarchyManager.class.getSimpleName();
+
+  // Debug mode is intentionally disabled (false). To enable, change to just ReactBuildConfig.DEBUG
+  @SuppressLint("ClownyBooleanExpression")
   private final boolean DEBUG_MODE = ReactBuildConfig.DEBUG && false;
 
   private final SparseArray<View> mTagsToViews;
@@ -248,7 +252,6 @@ private void updateInstanceHandle(View viewToUpdate, long instanceHandle) {
     viewToUpdate.setTag(R.id.view_tag_instance_handle, instanceHandle);
   }
 
-  @Nullable
   public synchronized long getInstanceHandle(int reactTag) {
     View view = mTagsToViews.get(reactTag);
     if (view == null) {
@@ -523,18 +526,6 @@ public synchronized void manageChildren(
     }
   }
 
-  private boolean arrayContains(@Nullable int[] array, int ele) {
-    if (array == null) {
-      return false;
-    }
-    for (int curEle : array) {
-      if (curEle == ele) {
-        return true;
-      }
-    }
-    return false;
-  }
-
   /**
    * Simplified version of constructManageChildrenErrorMessage that only deals with adding children
    * views
@@ -851,18 +842,6 @@ public synchronized void dispatchCommand(
     viewManager.receiveCommand(view, commandId, args);
   }
 
-  /**
-   * @return Themed React context for view with a given {@param reactTag} - it gets the context
-   *     directly from the view using {@link View#getContext}.
-   */
-  private ThemedReactContext getReactContextForView(int reactTag) {
-    View view = mTagsToViews.get(reactTag);
-    if (view == null) {
-      throw new JSApplicationIllegalArgumentException("Could not find view with tag " + reactTag);
-    }
-    return (ThemedReactContext) view.getContext();
-  }
-
   public synchronized void sendAccessibilityEvent(int tag, int eventType) {
     View view = mTagsToViews.get(tag);
     if (view == null) {