Skip to content

artillery - Add New Mission Type and Change Ammo Audio - #852

Merged
PabstMirror merged 4 commits into
masterfrom
artilleryTweaks
Mar 15, 2026
Merged

artillery - Add New Mission Type and Change Ammo Audio#852
PabstMirror merged 4 commits into
masterfrom
artilleryTweaks

Conversation

@lambdatiger

@lambdatiger lambdatiger commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

This PR:

  • Adds a faster lazy walk at tcvm's request
  • Changes to artillery round

@lambdatiger lambdatiger changed the title artillery - Add New Mission Type and Change Ammo Audion artillery - Add New Mission Type and Change Ammo Audio Mar 12, 2026
@lambdatiger
lambdatiger marked this pull request as ready for review March 12, 2026 22:39
@PabstMirror PabstMirror added this to the 1.3.73 milestone Mar 14, 2026
@PabstMirror
PabstMirror merged commit cf01c2c into master Mar 15, 2026
1 check passed
@PabstMirror
PabstMirror deleted the artilleryTweaks branch March 15, 2026 16:28
]
], _holdGuns] call CBA_fnc_targetEvent;
};
case ARTILLERY_MISSIONTYPE_SLOW_QUICK_LAZY_WALK: {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ref BrettMayson/HEMTT#1322

help[L-S52]: Duplicate case `11` in switch statement
    ┌─ addons/artillery/functions/fnc_beginMission.sqf:203:10
    │
177 │     case ARTILLERY_MISSIONTYPE_QUICK_LAZY_WALK: {
    │          -------------------------------------- first case here
    ·
203 │     case ARTILLERY_MISSIONTYPE_SLOW_QUICK_LAZY_WALK: {
    │          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ duplicate case

so I think this should be ARTILLERY_MISSIONTYPE_SLOW_QUICK_LAZY_BARRAGE?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

not the barrage there, I think the two cases can be collapsed into each other, but I'll look.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants