Page MenuHomeWildfire Games

Fixes includes and forward declarations in CPatchRData

Authored by vladislavbelov on Jun 3 2019, 10:30 PM.


Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
rP22371: Fixes includes and forward declarations in CPatchRData.

There were missed includes and forward declarations. But it was working because the CPatchRData.h was luckily included in a good place.

Test Plan
  1. Apply the patch and compile the game
  2. Make sure that it still compiles

Diff Detail

rP 0 A.D. Public Repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vladislavbelov created this revision.Jun 3 2019, 10:30 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build:

wraitii requested changes to this revision.Jun 5 2019, 9:32 PM
wraitii added a subscriber: wraitii.
wraitii added inline comments.
37 ↗(On Diff #8301)

This doesn't compile.

Need to add : short to this and the definition for me.

This revision now requires changes to proceed.Jun 5 2019, 9:32 PM
vladislavbelov added inline comments.Jun 5 2019, 9:40 PM
37 ↗(On Diff #8301)

I'd prefer to add include, because the storing type can be changed.

wraitii added inline comments.Jun 5 2019, 9:42 PM
37 ↗(On Diff #8301)


Fixes compilation.

vladislavbelov marked 2 inline comments as done.Jun 5 2019, 9:50 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build:

wraitii accepted this revision.Jun 6 2019, 9:32 PM

Compiles now indeed. Good cleanup, cheers.

This revision is now accepted and ready to land.Jun 6 2019, 9:32 PM
Stan added a subscriber: Stan.Jun 6 2019, 9:55 PM
Stan added inline comments.
31 ↗(On Diff #8328)

Won't that lead to redefinition ?

vladislavbelov added inline comments.Jun 6 2019, 9:57 PM
31 ↗(On Diff #8328)

Nope, it's declaration not definition.

Stan added inline comments.Jun 6 2019, 9:58 PM
31 ↗(On Diff #8328)

Yeah but it's already declared in the file you included so the forward declaration doesn't make sense, does it ?

vladislavbelov added inline comments.Jun 6 2019, 10:08 PM
31 ↗(On Diff #8328)

Yes, it makes sense in terms of the code support. You don't need to fix the code if you remove the only include.

This revision was automatically updated to reflect the committed changes.