Resolve minor issues

Author: Geolykt

src/main/java/de/geolykt/starloader/StarloaderAPIExtension.java

@@ -4,7 +4,7 @@
 
 import de.geolykt.starloader.api.Galimulator;
 import de.geolykt.starloader.api.event.EventManager;
-import de.geolykt.starloader.api.event.lifecycle.SignalExtensionTerminationExtension;
+import de.geolykt.starloader.api.event.lifecycle.SignalExtensionTerminationEvent;
 import de.geolykt.starloader.api.gui.Drawing;
 import de.geolykt.starloader.api.gui.modconf.ModConf;
 import de.geolykt.starloader.impl.DrawingManager;
@@ -30,7 +30,7 @@ public void preInitialize() {
 
     @Override
     public void unload() {
-        EventManager.handleEvent(new SignalExtensionTerminationExtension(this));
+        EventManager.handleEvent(new SignalExtensionTerminationEvent(this));
         getLogger().info("SLAPI is going to bed. Let's conquer the stars tomorrow!");
     }
 

src/main/java/de/geolykt/starloader/api/empire/Planet.java

@@ -5,7 +5,7 @@
 import org.jetbrains.annotations.NotNull;
 
 /**
- * @deprecated Planets are not stored in any meaningful matter as of know, obtaining them would be a pain.
+ * @deprecated Planets are not stored in any meaningful matter as of now; obtaining them would be a pain.
  * The full implementation of this interface will be delayed until the game changed in a way that this is possible.
  *
  * A planet that is part of a star system.
@@ -24,7 +24,7 @@ public interface Planet {
     /**
      * Obtains the name of the planet.
      * The game usually appends the ordinal (+1) of the planet to the name of the parent system
-     * to obtain the name of the planet. TODO can this also result in other names?
+     * to obtain the name of the planet.
      *
      * @return The name of the planet.
      */

…SignalExtensionTerminationExtension.java …cle/SignalExtensionTerminationEvent.javasrc/main/java/de/geolykt/starloader/api/event/lifecycle/SignalExtensionTerminationExtension.java renamed to src/main/java/de/geolykt/starloader/api/event/lifecycle/SignalExtensionTerminationEvent.java src/main/java/de/geolykt/starloader/api/event/lifecycle/SignalExtensionTerminationExtension.java renamed to src/main/java/de/geolykt/starloader/api/event/lifecycle/SignalExtensionTerminationEvent.java

@@ -11,10 +11,10 @@
  * Due to how SLL Classloading works, the entirety of the SLAPI becomes unavailable after this event is
  * fired with a {@link StarloaderAPIExtension} as the targeted extension.
  */
-public class SignalExtensionTerminationExtension extends Event {
+public class SignalExtensionTerminationEvent extends Event {
     protected final Extension unloadingExtension;
 
-    public SignalExtensionTerminationExtension(Extension unloadingExtension) {
+    public SignalExtensionTerminationEvent(Extension unloadingExtension) {
         this.unloadingExtension = unloadingExtension;
     }
 

src/main/java/de/geolykt/starloader/api/gui/InputDialog.java

@@ -3,6 +3,7 @@
 import java.util.function.Consumer;
 
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 
 public interface InputDialog extends Closable {
 
@@ -14,7 +15,7 @@ public interface InputDialog extends Closable {
      *
      * @param hook The Hook to attach
      */
-    public void addHook(@NotNull Consumer<String> hook);
+    public void addHook(@NotNull Consumer<@Nullable String> hook);
 
     /**
      * Closes the object from view. However unlike {@link #close()} this method does

src/main/java/de/geolykt/starloader/impl/StarloaderInputDialog.java

@@ -3,6 +3,7 @@
 import java.util.function.Consumer;
 
 import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
 
 import de.geolykt.starloader.api.gui.InputDialog;
 
@@ -19,7 +20,7 @@ public StarloaderInputDialog(String title, TextInputWrapper wrapper, String text
     }
 
     @Override
-    public void addHook(@NotNull Consumer<String> hook) {
+    public void addHook(@NotNull Consumer<@Nullable String> hook) {
         this.wrapper.addHook(hook);
     }
 

src/main/java/de/geolykt/starloader/impl/actors/WrappedActor.java

@@ -3,6 +3,7 @@
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
+import de.geolykt.starloader.api.NullUtils;
 import de.geolykt.starloader.api.actor.ActorSpec;
 import de.geolykt.starloader.api.actor.wrapped.WrappingActor;
 import de.geolykt.starloader.api.actor.wrapped.WrappingConfiguration;
@@ -44,10 +45,7 @@ public float getMaximumVelocity() {
         if (config.inheritName()) {
             return delegate.getName();
         }
-        String name = super.getName();
-        if (name == null) {
-            throw new NullPointerException("The name was null!");
-        }
+        String name = NullUtils.requireNotNull(super.getName());
         return name;
     }
 

src/main/java/de/geolykt/starloader/impl/actors/WrappedMissile.java

@@ -3,6 +3,7 @@
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
+import de.geolykt.starloader.api.NullUtils;
 import de.geolykt.starloader.api.actor.ActorSpec;
 import de.geolykt.starloader.api.actor.spacecrafts.MissileSpec;
 import de.geolykt.starloader.api.actor.wrapped.WrappingActor;
@@ -58,11 +59,7 @@ public float getMaximumVelocity() {
         if (config.inheritName()) {
             return delegate.getName();
         }
-        String name = super.getName();
-        if (name == null) {
-            throw new NullPointerException("The name of the missile is somehow null!");
-        }
-        return name;
+        return NullUtils.requireNotNull(super.getName());
     }
 
     @Override