unify(common-rts): Move Common/RTS to Core - #3048
Conversation
|
ResourceGatheringManager.cpp needs to have a merge first (separate commit) before being merged. |
|
How about merge all the Common/RTS files first, then move them all? |
I will revise this PR when I have the opportunity. |
edebd96 to
7407abb
Compare
7407abb to
7bdb298
Compare
I’ve rebuilt this PR, and for now there’s only one commit. The first commit simply merges the files so that Generals and Zero Hour use the same code. What I’m unsure about is whether I’ve implemented the CRC gating correctly, or if I’m even heading in the right direction with it. (I doubt it) I’ve also put comments and dead code behind CRC gates to at least distinguish which source they came from. That should make a later cleanup easier, although I know that cleanup is out of scope for this PR. I’ll do the actual unification commit and update the PR body once I get some direction on whether I’m heading in the right direction. |
This moves the remaining shared low-risk Common RTS files into Core:
The moved files are code-identical between both games except for the product-name license banner. ResourceGatheringManager.cpp also corrects a comment spelling.
Verification