Skip to content

tweak(worldbuilder): Remove EULA code and data#2307

Open
Caball009 wants to merge 3 commits intoTheSuperHackers:mainfrom
Caball009:remove_world_builder_eula_window
Open

tweak(worldbuilder): Remove EULA code and data#2307
Caball009 wants to merge 3 commits intoTheSuperHackers:mainfrom
Caball009:remove_world_builder_eula_window

Conversation

@Caball009
Copy link

@Caball009 Caball009 commented Feb 15, 2026

image

Currently every launch of the (Zero Hour) World Builder requires accepting the end-user license agreement. This is unnecessarily inconvenient. The Steam version of the WB doesn't show this window, and it was pointed out to me that we're not required to show this window under GPL v3.0. This PR should remove all code and data relating to the EULA window.

TODO:

  • Replicate in Generals.

@Caball009 Caball009 added Enhancement Is new feature or request Minor Severity: Minor < Major < Critical < Blocker WorldBuilder Relates to World Builder labels Feb 15, 2026
@Caball009 Caball009 changed the title tweak: Remove world builder EULA code and data tweak: Remove World Builder EULA code and data Feb 15, 2026
@Caball009 Caball009 changed the title tweak: Remove World Builder EULA code and data tweak(worldbuilder): Remove EULA code and data Feb 15, 2026
@greptile-apps
Copy link

greptile-apps bot commented Feb 15, 2026

Greptile Overview

Greptile Summary

Removed the EULA dialog that appeared on every Zero Hour World Builder launch, improving user experience by eliminating unnecessary interruption.

  • Deleted EulaDialog.cpp and euladialog.h files
  • Removed EULA dialog invocation from InitInstance() in WorldBuilder.cpp
  • Removed EULA dialog resource and string definitions from WorldBuilder.rc
  • Updated CMakeLists.txt to remove EULA files from build sources

The changes are clean and straightforward. Minor cleanup needed: leftover resource definitions in resource.h should be removed for completeness.

Confidence Score: 4/5

  • Safe to merge with minor cleanup recommended
  • The PR successfully removes all EULA code and data from the GeneralsMD World Builder. The changes are straightforward: file deletions, removal of function calls, and resource cleanup. However, unused resource definitions remain in resource.h (7 constants that should be deleted). This is a minor cleanup issue that doesn't affect functionality but should be addressed for code hygiene.
  • Consider cleaning up GeneralsMD/Code/Tools/WorldBuilder/res/resource.h to remove leftover EULA-related constants

Important Files Changed

Filename Overview
GeneralsMD/Code/Tools/WorldBuilder/res/WorldBuilder.rc Removed EULA dialog resource and all EULA text string resources
GeneralsMD/Code/Tools/WorldBuilder/src/WorldBuilder.cpp Removed include of euladialog.h and removed EULA dialog modal call from InitInstance()

Flowchart

flowchart TD
    A[World Builder Launch] --> B{BEFORE: EULA Dialog}
    B -->|Accept| C[Initialize Application]
    B -->|Cancel| D[Exit Application]
    C --> E[World Builder Runs]
    
    F[World Builder Launch] --> G[AFTER: Direct Initialization]
    G --> H[World Builder Runs]
    
    style B fill:#ffcccc
    style D fill:#ffcccc
    style G fill:#ccffcc
    style H fill:#ccffcc
Loading

Last reviewed commit: e5b11af

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

5 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@@ -1693,17 +1693,6 @@ BEGIN
BS_AUTOCHECKBOX | BS_LEFTTEXT | WS_TABSTOP,40,51,95,14
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leftover EULA resource definitions remain in resource.h: IDD_EULA_AGREEMENT (line 134), IDC_EDIT1 (line 379), and IDS_EULA_AGREEMENT1-5 (lines 752-756). These should be removed.

Prompt To Fix With AI
This is a comment left during a code review.
Path: GeneralsMD/Code/Tools/WorldBuilder/res/WorldBuilder.rc
Line: 1693:1693

Comment:
Leftover EULA resource definitions remain in `resource.h`: `IDD_EULA_AGREEMENT` (line 134), `IDC_EDIT1` (line 379), and `IDS_EULA_AGREEMENT1-5` (lines 752-756). These should be removed.

How can I resolve this? If you propose a fix, please make it concise.

@Caball009 Caball009 added the Refactor Edits the code with insignificant behavior changes, is never user facing label Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Is new feature or request Minor Severity: Minor < Major < Critical < Blocker Refactor Edits the code with insignificant behavior changes, is never user facing WorldBuilder Relates to World Builder

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant