Revert D48588377: EZ cleanup of TurboReactPackage

Differential Revision:
D48588377

Original commit changeset: 5645acad938a

Original Phabricator Diff: D48588377

fbshipit-source-id: e0da38e5b721d6bfc7fd6b4607c7712301330a8f

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/TurboReactPackage.java

@@ -7,6 +7,7 @@
 
 package com.facebook.react;
 
+import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import com.facebook.react.bridge.ModuleHolder;
 import com.facebook.react.bridge.ModuleSpec;
@@ -38,8 +39,9 @@ public List<NativeModule> createNativeModules(ReactApplicationContext reactConte
    * The API needed for TurboModules. Given a module name, it returns an instance of {@link
    * NativeModule} for the name
    *
-   * @param name name of the Native Module
-   * @param reactContext {@link ReactApplicationContext} context for this
+   * @param name
+   * @param reactContext
+   * @return
    */
   public abstract @Nullable NativeModule getModule(
       String name, final ReactApplicationContext reactContext);
@@ -58,10 +60,13 @@ Iterable<ModuleHolder> getNativeModuleIterator(final ReactApplicationContext rea
     final Set<Map.Entry<String, ReactModuleInfo>> entrySet =
         getReactModuleInfoProvider().getReactModuleInfos().entrySet();
     final Iterator<Map.Entry<String, ReactModuleInfo>> entrySetIterator = entrySet.iterator();
-    // This should ideally be an IteratorConvertor, but we don't have any internal library for it
-    return () ->
-        new Iterator<ModuleHolder>() {
-          @Nullable Map.Entry<String, ReactModuleInfo> nextEntry = null;
+    return new Iterable<ModuleHolder>() {
+      @NonNull
+      @Override
+      // This should ideally be an IteratorConvertor, but we don't have any internal library for it
+      public Iterator<ModuleHolder> iterator() {
+        return new Iterator<ModuleHolder>() {
+          Map.Entry<String, ReactModuleInfo> nextEntry = null;
 
           private void findNext() {
             while (entrySetIterator.hasNext()) {
@@ -113,6 +118,8 @@ public void remove() {
             throw new UnsupportedOperationException("Cannot remove native modules from the list");
           }
         };
+      }
+    };
   }
 
   /**