diff options
| author | TheKodeToad <TheKodeToad@proton.me> | 2022-11-03 16:40:23 +0000 |
|---|---|---|
| committer | TheKodeToad <TheKodeToad@proton.me> | 2022-11-08 16:35:43 +0000 |
| commit | 8ce78dcc54a8ed3e6292a8650692fa9c520a993d (patch) | |
| tree | b1b188f289728d019dc259243b5c296cb1eba982 /libraries/launcher/org/prismlauncher/utils | |
| parent | afe088dba18b83ae1ea79e3b31ef5026f68b7b6d (diff) | |
| download | PrismLauncher-8ce78dcc54a8ed3e6292a8650692fa9c520a993d.tar.gz PrismLauncher-8ce78dcc54a8ed3e6292a8650692fa9c520a993d.tar.bz2 PrismLauncher-8ce78dcc54a8ed3e6292a8650692fa9c520a993d.zip | |
Try to improve consistency
- Makes code formatting more consistent with the C++ codebase. Probably removes some trailing whitespace. Maybe it would be best to commit an Eclipse or IntelliJ code format preferences file?
- Removes obscure suppressions. I personally think it's better to only suppress warnings that javac complains about. Suppressing a lot of non-standardised warnings (many of them turned off by default even in IntelliJ) just creates needless clutter.
- Fixes some trivial warnings instead of suppressing them. serialVersionUID is sort of stupid, but I'd rather mentally ignore it or just fix it if it's really that annoying.
Signed-off-by: TheKodeToad <TheKodeToad@proton.me>
Diffstat (limited to 'libraries/launcher/org/prismlauncher/utils')
| -rw-r--r-- | libraries/launcher/org/prismlauncher/utils/ReflectionUtils.java | 9 | ||||
| -rw-r--r-- | libraries/launcher/org/prismlauncher/utils/StringUtils.java | 5 |
2 files changed, 7 insertions, 7 deletions
diff --git a/libraries/launcher/org/prismlauncher/utils/ReflectionUtils.java b/libraries/launcher/org/prismlauncher/utils/ReflectionUtils.java index b0eefec7..ad9e57fd 100644 --- a/libraries/launcher/org/prismlauncher/utils/ReflectionUtils.java +++ b/libraries/launcher/org/prismlauncher/utils/ReflectionUtils.java @@ -49,8 +49,8 @@ import java.lang.reflect.Modifier; import java.util.logging.Level; import java.util.logging.Logger; - public final class ReflectionUtils { + private static final Logger LOGGER = Logger.getLogger("ReflectionUtils"); private ReflectionUtils() { @@ -160,10 +160,9 @@ public final class ReflectionUtils { * @throws NoSuchMethodException If no method matching the correct signature can be found * @throws IllegalAccessException If method handles cannot access the entrypoint */ - public static MethodHandle findMainEntrypoint(String entrypointClassName) throws - ClassNotFoundException, - NoSuchMethodException, - IllegalAccessException { + public static MethodHandle findMainMethod(String entrypointClassName) + throws ClassNotFoundException, NoSuchMethodException, IllegalAccessException { return findMainEntrypoint(ClassLoader.getSystemClassLoader().loadClass(entrypointClassName)); } + } diff --git a/libraries/launcher/org/prismlauncher/utils/StringUtils.java b/libraries/launcher/org/prismlauncher/utils/StringUtils.java index d470624c..a371b0cb 100644 --- a/libraries/launcher/org/prismlauncher/utils/StringUtils.java +++ b/libraries/launcher/org/prismlauncher/utils/StringUtils.java @@ -36,8 +36,8 @@ package org.prismlauncher.utils; - public final class StringUtils { + private StringUtils() { } @@ -46,6 +46,7 @@ public final class StringUtils { if (splitPoint == -1) return null; - return new String[]{ input.substring(0, splitPoint), input.substring(splitPoint + 1) }; + return new String[] { input.substring(0, splitPoint), input.substring(splitPoint + 1) }; } + } |
