refactor: Add function for streaming bay weapons

WeaponMounted.getBayWeapons() iterates through the entire bay before
returning, which isn't always necessary. It has been refactored to allow
for the stream to be used directly.

Author: Saklad5

megamek/src/megamek/common/equipment/WeaponMounted.java

@@ -30,6 +30,7 @@
 import java.util.Objects;
 import java.util.Vector;
 import java.util.stream.Collectors;
+import java.util.stream.Stream;
 
 public class WeaponMounted extends Mounted<WeaponType> {
 
@@ -207,12 +208,19 @@ public void clearBayWeapons() {
     }
 
     /**
-     * @return All the weapon mounts in the bay.
+     * @return A stream containing the weapon mounts in the bay.
      */
-    public List<WeaponMounted> getBayWeapons() {
+    public Stream<WeaponMounted> streamBayWeapons() {
         return bayWeapons.stream()
                 .map(i -> getEntity().getWeapon(i))
-                .filter(Objects::nonNull)
+                .filter(Objects::nonNull);
+    }
+
+    /**
+     * @return All the weapon mounts in the bay.
+     */
+    public List<WeaponMounted> getBayWeapons() {
+        return streamBayWeapons()
                 .collect(Collectors.toList());
     }