Differential D1429

Fix cheatTimeMultiplier broken by rP21423
ClosedPublic
Actions

Authored by mimo on Apr 2 2018, 5:01 PM.

Details

Reviewers

temple

Group Reviewers

Restricted Owners Package

(Owns No Changed Paths)

Commits

rP21641: Fix cheatTimeMultiplier broken by rP21423

Summary

As the title say

Test Plan

Check that the cheat now works

Diff Detail

Repository

rP 0 A.D. Public Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

mimo created this revision.Apr 2 2018, 5:01 PM

mimo mentioned this in rP21423: Add a time multiplier for AI levels.Apr 2 2018, 5:03 PM

Tests okay. I think only the gather rate multiplier and cheat time multiplier are relevant with respect to resource gatherer, but I guess it can't hurt to do time multiplier too.

This revision is now accepted and ready to land.Apr 2 2018, 7:01 PM

In D1429#58364, @temple wrote:

Tests okay. I think only the gather rate multiplier and cheat time multiplier are relevant with respect to resource gatherer, but I guess it can't hurt to do time multiplier too.

Time could also be relevant as it has the same use as cheatTime without cheating (althought it is currently only used by the ai for its difficulty levels).

Closed by commit rP21641: Fix cheatTimeMultiplier broken by rP21423 (authored by mimo). · Explain WhyApr 2 2018, 10:30 PM

This revision was automatically updated to reflect the committed changes.

(Likely unrelated, but maybe not unrelated. I might check tomorrow #5113)