Skip to content

Add command-line option overrides to options system#7561

Open
The-E wants to merge 3 commits into
scp-fs2open:masterfrom
The-E:cmdline-priority-rework
Open

Add command-line option overrides to options system#7561
The-E wants to merge 3 commits into
scp-fs2open:masterfrom
The-E:cmdline-priority-rework

Conversation

@The-E

@The-E The-E commented Jul 4, 2026

Copy link
Copy Markdown
Member
  • Extend OptionsManager to support storing and displaying reasons for overrides.
  • Update related code paths to prioritize command-line overrides, ensure correct option resolution, and display override details in the UI.
  • Add unit tests for override behavior and round-trip validation.
  • Remove unused legacy functions (removeVSyncOption, removeWindowModeOption) in favor of the new system.

The-E added 3 commits July 21, 2026 19:29
- Extend `OptionsManager` to support storing and displaying reasons for overrides.
- Update related code paths to prioritize command-line overrides, ensure correct option resolution, and display override details in the UI.
- Add unit tests for override behavior and round-trip validation.
- Remove unused legacy functions (`removeVSyncOption`, `removeWindowModeOption`) in favor of the new system.
…ger` overrides

- Update HDR, shadow rendering, and graphics API option overrides to include their respective command-line flag names.
- Refactor `Graphics.RenderAPI` to use the `GraphicsAPI` enum instead of raw integers for better type safety and clarity.
- Ensure Vulkan support logic and fallback behavior function correctly with the updated options system.
- Update unit tests to validate override functionality and enum-based API configuration.
@The-E
The-E force-pushed the cmdline-priority-rework branch from 76203e4 to ebb5d3c Compare July 21, 2026 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant