fix: remove duplicate --width/--height for Forge - version.json already includes them via resolution_width/resolution_height placeholders
This commit is contained in:
+3
-9
@@ -151,15 +151,9 @@ public class LaunchCommandBuilder {
|
|||||||
command.addAll(getModloaderLaunchArgs());
|
command.addAll(getModloaderLaunchArgs());
|
||||||
}
|
}
|
||||||
|
|
||||||
// Append width/height if custom resolution
|
// NOTE: --width/--height are NOT appended here because version.json
|
||||||
if (options.getWidth() > 0) {
|
// game args already include them via ${resolution_width}/${resolution_height}
|
||||||
command.add("--width");
|
// placeholders. Appending them again causes MultipleArgumentsForOptionException.
|
||||||
command.add(String.valueOf(options.getWidth()));
|
|
||||||
}
|
|
||||||
if (options.getHeight() > 0) {
|
|
||||||
command.add("--height");
|
|
||||||
command.add(String.valueOf(options.getHeight()));
|
|
||||||
}
|
|
||||||
} else {
|
} else {
|
||||||
// Fabric, vanilla, and other loaders: use manual JVM args
|
// Fabric, vanilla, and other loaders: use manual JVM args
|
||||||
command.addAll(getJvmArguments(options));
|
command.addAll(getJvmArguments(options));
|
||||||
|
|||||||
Reference in New Issue
Block a user