Performance: Refactored Engine::DoNextAction

[nl-saw]

Refactored Engine::DoNextAction so it is more performance-oriented. Mainly it applies multipliers early, which optimizes the action processing flow, and it exits the loop more efficiently when there’s no action to process.

I'd appreciate if some of you can test it to make sure there are no deficiencies in bot behavior.

No rush in merging :)

[blinkysc]

Putting on mine now

[blinkysc]

It seems stable for me. I still get sharp spikes, with the max percentiles reaching up to around 8,000 when using .server info, but this has been the status quo for quite some time.

[nl-saw]

It seems stable for me. I still get sharp spikes, with the max percentiles reaching up to around 8,000 when using .server info, but this has been the status quo for quite some time.

Thanks for the feedback, this is just a minor performance tune and will not help much with (huge) spikes.
Reason to test is to make sure the bot's behaviors are still intact.

[blinkysc]

It seems stable for me. I still get sharp spikes, with the max percentiles reaching up to around 8,000 when using .server info, but this has been the status quo for quite some time.

Thanks for the feedback, this is just a minor performance tune and will not help much with (huge) spikes. Reason to test is to make sure the bot's behaviors are still intact.

Everything has been same so far for bots. I am using new RPG strat though too, though not sure if that would be a big diff

[nl-saw]

Everything has been same so far for bots. I am using new RPG strat though too, though not sure if that would be a big diff

Thanks and no, bots use the do next action regardless of chosen strat.