Maybe I could just set unit to uncontrollable @Freagarach ? Since 1960 got merged.

Ah I messed that sentence up in all ways possible.
Should be: since only infantry can garrison in barracks now.

*cavalry

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

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

Since the patch is about timing i don't see how to test it without introducing a race condition.

uint32_t changed to u32

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

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

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

Add some includes, use timer_Time.

I don't think the condition is useful without context. But i'm ok if i should leave it that way.

In D4761#202748, @phosit wrote:

it does compile on all os. Ther might be some in architecture specivic code but a grep yields nothing.

In D4761#202710, @vladislavbelov wrote:

Have you checked MSVC output in case of an error?

Reduce simulation thread load by ~10% (maybe a bit less, can't think of a way to test) and network traffic by 10-18%, depending on player APM. 500ms indeed sounds like too much. But 200 might have been an over-correction. The difference from 2 turns per second to 5 turns per second is ~1/3 second, but the difference between 5 and 4 is 1/20 second. I think this is small enough to not make a big difference.

Another question is if this should also remove these functions itself as well, to not stack dead code

Gotta check what this do to translations.

Have you checked MSVC output in case of an error?

I suggest to add some information explaining user what's wrong, they won't understand what it means

If SM allows to get a size of currently allocated memory it should be easy to write a unit test. We already have some unit tests for ScriptInterface.