Add Timeout CBA_fnc_waitUntilAndExecute

[654wak654]

When merged this pull request will:

• Title
• Add _timeout and _timeoutCode params, put calls that use them in a new GVAR(waitUntilAndExecTimeoutArray) array to not add any overhead to the usual GVAR(waitUntilAndExecArray).

Separating the calls with timeout and without timeout into different arrays seemed like a better option then to add an overhead to everything that currently uses CBA's main PFH.

[commy2]

could move this line after L81, so the flag is only reset if it ever where changed to true. I think the same can be done above that once too. Minor performance improvement.

[commy2]

maybe use isNil "_timeout" and have no default value / default value nil in params?
No one will ever use 1E9, but having a random high number seems weird. (Especially an integer above single point floating precision.)

[PabstMirror]

How about default timeout value of -1,
and only use time if _timeout >= 0

[654wak654]

Could make default 0 and go for _timeout > 0, since 0 whould mean there is an immediate timeout.

[PabstMirror]

I think we want CBA_missionTime instead of time

[654wak654]

I used time because it's to measure relative time on one machine, but I can change it.

[PabstMirror]

Using this wrapper, we can use the existing waitUntilAndExecArray, and make no changes to init_perFrameHandler.sqf. So there would be no per-frame overhead.

if (_timeout == 1E9) then {
    GVAR(waitUntilAndExecArray) pushBack [_condition, _statement, _args];
} else {
    GVAR(waitUntilAndExecArray) pushBack [{
        params ["_condition", "_statement", "_args", "_timeout", "_timeoutCode"];

        if (CBA_missionTime > _timeout) exitWith {
            systemChat "timeout";
            _args call _timeoutCode;
            true
        };
        if (_args call _condition) exitWith {
            systemChat "condition";
            _args call _statement;
            true
        };
        false
    }, {}, [_condition, _statement, _args, _timeout, _timeoutCode, CBA_missionTime]];
};

[654wak654]

The timeout condition is CBA_missionTime - _startTime > _timeout, but other than that your code works great Pabst. Also change default timeout to 0 and did that """optimization""" with _delete.

[commy2]

lgtm