Skip to content

Add desktop launcher tuning flags and modernization roadmap#1

Merged
awest813 merged 1 commit into
mainfrom
codex/analyze-game-ngien-for-modernization
Mar 2, 2026
Merged

Add desktop launcher tuning flags and modernization roadmap#1
awest813 merged 1 commit into
mainfrom
codex/analyze-game-ngien-for-modernization

Conversation

@awest813
Copy link
Copy Markdown
Owner

@awest813 awest813 commented Mar 2, 2026

Motivation

  • Provide runtime knobs to improve compatibility and performance without changing code by exposing common launcher options.
  • Make the launcher more discoverable and easier to test for different GPU/OS configurations by adding a -help command and windowed/quality flags.
  • Capture a practical modernization roadmap to guide follow-up work across modernization, compatibility, performance, and QoL.

Description

  • Extended DesktopLauncher to accept and apply new CLI options including -help/--help/-h, -windowed [width height] (alias -window preserved), -vsync/-novsync, -fps <value>, and -msaa <samples>, with defaults and safe parsing fallbacks.
  • Added DEFAULT_WINDOWED_WIDTH/DEFAULT_WINDOWED_HEIGHT constants and utility methods isInteger and parseIntWithFallback to validate numeric arguments and print fallback warnings.
  • Wired parsed values into LwjglApplicationConfiguration (width, height, fullscreen, vSyncEnabled, samples, foregroundFPS, backgroundFPS) so users can tune MSAA, FPS caps, and VSync from the launcher.
  • Updated README.md to document the new launcher flags and added docs/ENGINE_MODERNIZATION_ANALYSIS.md containing a phased roadmap for modernization, compatibility, performance, and QoL improvements.

Testing

  • Attempted to compile the desktop module with ./gradlew :desktop:compileJava to validate the change in this environment, but the build failed due to a local Gradle/Groovy vs JDK mismatch (Unsupported class file major version 65) unrelated to the launcher logic.
  • No other automated tests were executed in this environment; the change is intentionally low-risk and limited to argument parsing and launcher wiring.

Codex Task

@awest813 awest813 merged commit 778f9f5 into main Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant