Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add failure code argument to fnc_progressBar #1177

Merged
merged 1 commit into from
Jul 7, 2019
Merged

Conversation

McDiod
Copy link
Contributor

@McDiod McDiod commented Jul 7, 2019

When merged this pull request will:

  • add a failure code argument to fnc_progressBar's onSuccess and onFailure calls

@commy2 commy2 added this to the 3.12 milestone Jul 7, 2019
@commy2 commy2 merged commit a1132ec into CBATeam:master Jul 7, 2019
@@ -75,7 +80,7 @@ if (!isNil QGVAR(ProgressBarParams)) then {
GVAR(ProgressBarParams) params ["_arguments", "", "", "_onFailure", "_startTime", "_totalTime"];
private _elapsedTime = (CBA_missionTime - _startTime) min _totalTime;

[_onFailure, [_arguments, false, _elapsedTime, _totalTime]] call CBA_fnc_execNextFrame;
[_onFailure, [_arguments, false, _elapsedTime, _totalTime, 3]] call CBA_fnc_execNextFrame;
Copy link
Contributor

@Dystopian Dystopian Jul 8, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these magic numbers should be macro

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants