Skip to content

merge: sync with TheSuperHackers upstream - 2026-06-08#158

Merged
fbraz3 merged 5 commits into
mainfrom
thesuperhackers-sync-06-08-2026
Jun 9, 2026
Merged

merge: sync with TheSuperHackers upstream - 2026-06-08#158
fbraz3 merged 5 commits into
mainfrom
thesuperhackers-sync-06-08-2026

Conversation

@fbraz3

@fbraz3 fbraz3 commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Description

Sync with upstream TheSuperHackers/GeneralsGameCode — imports 4 upstream commits without regressing the GeneralsX cross-platform architecture.

Upstream Changes Merged

Commit Description
deb728c0f perf(gamemessage) — Replace GameMessageArgument linked lists with std::vector for O(1) indexed access
98f1db992 unify(ww3d2) — Merge and clean up SortingRenderer code
a1358a97e unify(ww3d2) — Move SortingRenderer to Core/Libraries/Source/WWVegas/WW3D2/
0e7f686e8 refactor(gamelogic) — Split logicMessageDispatcher switch cases into ~40 individual handler methods

Conflict Resolution

1 conflict resolved: sortingrenderer.cpp (modify/delete) — accepted upstream deletion (file moved to Core/). No platform-specific logic was lost.

All auto-merged changes are internal engine refactors that do not touch platform abstraction layers (SDL3, DXVK, OpenAL, FFmpeg).

Build Validation

Target Platform Result
GeneralsXZH macOS (Vulkan) Built successfully (1460/1460)
GeneralsX macOS (Vulkan) Built successfully (785/785)

Risk Areas

  • GameLogicDispatch.cpp — large refactor, should be reviewed for correctness
  • NetCommandMsg.h/cpp and MessageStream.h/cpp — vector conversion changes memory ownership model

@fbraz3 fbraz3 changed the title Sync with TheSuperHackers upstream (2026-06-08) merge: sync with TheSuperHackers upstream - 2026-06-08 Jun 9, 2026
@fbraz3 fbraz3 merged commit 1280a1f into main Jun 9, 2026
8 checks passed
@fbraz3 fbraz3 deleted the thesuperhackers-sync-06-08-2026 branch June 9, 2026 03:08
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.

4 participants