Map events #60/#61 are implemented very inefficiently

[DCoder DCoder]

These two events use highly inefficient logic to reach their goals. Every frame they recalculate which of the 600+ TechnoTypes is in the condition (waste of time - this answer is not going to change in the middle of the game) and then count the instances of that TechnoType in the global TechnoClass::Array .

Changing them to precompute the wanted conditional TechnoType at match start and then summing the amount of that type each HouseClass has would bring this down from a waste of time to a compact and acceptably fast check.

[Chanterier (speederyr)]

Well, they do seem to work. I'm going to do more tests for lag though.

[AlexB (alexander-b)]

The fix is not official yet. The binary was just one of my debug builds.

[AlexB (alexander-b)]

Now it's official. The v02 branch, aka Fatman, contains two replacement implementations for both events. If you notice problems using these tags, or they don't behave as always, please notify me.

Event 61 is used in the first missions of both the allied and Soviet campaigns.

[Chanterier (speederyr)]

TechType Does Not Exist definitely works.

[Chanterier (speederyr)]

TechType Exists also works.

[AlexB (alexander-b)]

Thanks for testing, Speeder!