Account for child EventBuses in dispose() and trim() (#83)

Author: PaintNinja

src/main/java/net/minecraftforge/eventbus/internal/AbstractEventBusImpl.java

@@ -141,8 +141,9 @@ default void startup() {
             // Force invalidate the invoker to remove the no-op invoker that might've been set by shutdown()
             alreadyInvalidated().set(false);
             invalidateInvoker();
+
+            children().forEach(AbstractEventBusImpl::startup);
         }
-        children().forEach(AbstractEventBusImpl::startup);
     }
 
     default void shutdown() {
@@ -154,8 +155,9 @@ default void shutdown() {
             // on calls to addListener() to keep the no-op invoker
             setNoOpInvoker();
             alreadyInvalidated().set(true);
+
+            children().forEach(AbstractEventBusImpl::shutdown);
         }
-        children().forEach(AbstractEventBusImpl::shutdown);
     }
 
     default void dispose() {
@@ -166,14 +168,22 @@ default void dispose() {
 
             backingList().trimToSize();
             monitorBackingList().trimToSize();
+
+            children().forEach(AbstractEventBusImpl::dispose);
+
+            if (children() instanceof ArrayList<?> childrenArrayList) {
+                childrenArrayList.clear();
+                childrenArrayList.trimToSize();
+            }
         }
-        children().forEach(AbstractEventBusImpl::dispose);
     }
 
     default void trim() {
         synchronized (backingList()) {
             backingList().trimToSize();
             monitorBackingList().trimToSize();
+            if (children() instanceof ArrayList<?> childrenArrayList)
+                childrenArrayList.trimToSize();
         }
     }