Improve UnitBuilderBotModule and BaseBuilderBotModule multi-queue performance

[dnqbob]

UnitBuilderBotModule and BaseBuilderBotModule fix on muti-queue performance:

1. Only allow new item being queued when cash above a certain number

2. Only tick one kind of queues at one tick, reduce the pressure on the actived tick

3. 'BaseBuilderBotModule' will check all buildings in producing, avoid queue mutiple same buildings.

About to make a testcase in cnc mod

[dnqbob]

Note: the AI test case here is the way I making AI for SP and RV.

I add a small cash cheat for AI's Conyers so as AI build more sub-base, they get more resources like human player expand their base for more resources.

this also prevent AI get oneshot by superweapon, making new player have to learn to expand their base and forcus more on squad to win the AI game

[PunkPun]

Partial review

[dnqbob]

AI test has problem, will fix it later

[dnqbob]

Should be all fixed, thanks for the reviews.

[RoosterDragon]

Some small fixups, but otherwise LGTM.

[PunkPun]

About 2 minutes into a RA game all bots stopped producing anything, no units nor structures

[dnqbob]

Bug confirmed, it is strange that only cnc is normal. Will fix it later.

[dnqbob]

Ok guys, I spoted a wierd bug when I am debugging. I tried output the cash the module used for check, but found the cash get by the module is completely different from the actual cash:

So this is the reason that AI stop producing.

This cash bug happens when the first time AI spend all the cash, and from then on the cash get by module is always 0. I guess it is not the problem of the module here, it should be a commit after this PR broke the related code.

I add a commit to show cash when module check cash.

@PunkPun

[PunkPun]

Could it be that it's using cash instead of resources and cash?

[dnqbob]

it is possible.

In addition, We have

(baseBuilder.Info.NewProductionCashThreshold > 0 && playerResources.Resources > baseBuilder.Info.NewProductionCashThreshold

At this place we only use Resources instead of Resources and Cash? Is this place also a problem?

[dnqbob]

Add GetCashAndResources() to make the code more understandable on where use Cash + Resources.

Also fix NewProductionCashThreshold only use resources

[PunkPun]

LGTM

GetCashAndResources() will be superseded by the pending resource refactor, though it will be easy to deprecate

[PunkPun]

Changelog