BrokenRegistry / Rotp-Fusion

Other
22 stars 4 forks source link

Bring back old normalPopGrowth() functionality via Planet #6

Closed dHannasch closed 1 year ago

dHannasch commented 1 year ago

Ah, hmm, looks like https://github.com/BrokenRegistry/Rotp-Fusion/commit/45e2d1e0b23ac129e7e56774dddd0fd61176789a just removed the normalPopGrowth(int toSend) that https://github.com/BrokenRegistry/Rotp-Fusion/pull/5 uses. I'm assuming you have your reasons for not wanting it in Colony, but what about allowing Planet to do something similar?

Making a separate pull request for this to make it easier to see that this doesn't change any functionality (this is a very dangerous operation, since normalPopGrowth() is the actual function that determines real population growth when nextTurn() is called). This solely relocates the code and has the Colony just pass its population (thereby allowing passing a different, expected population).