Skip to content

Minedetector - Remove unused param and format code#11259

Merged
PabstMirror merged 1 commit intomasterfrom
minedetector-minor-fixes
Jan 28, 2026
Merged

Minedetector - Remove unused param and format code#11259
PabstMirror merged 1 commit intomasterfrom
minedetector-minor-fixes

Conversation

@johnb432
Copy link
Contributor

When merged this pull request will:

  • Title.

IMPORTANT

  • If the contribution affects the documentation, please include your changes in this pull request so the documentation will appear on the website.
  • Development Guidelines are read, understood and applied.
  • Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

@johnb432 johnb432 added this to the 3.20.3 milestone Jan 27, 2026
@johnb432 johnb432 added kind/cleanup Release Notes: **CHANGED:** ignore-changelog Release Notes: Excluded labels Jan 27, 2026
@PabstMirror PabstMirror merged commit 7d113ef into master Jan 28, 2026
3 of 4 checks passed
@PabstMirror PabstMirror deleted the minedetector-minor-fixes branch January 28, 2026 20:34
* Public: Yes
*/

params [["_unit", objNull, [objNull]], ["_detectorType", "", [""]]];
Copy link
Member

@jokoho48 jokoho48 Jan 28, 2026

Choose a reason for hiding this comment

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

this is a public function where the header says still detector type as 2nd parameter. not sure if this was more of a bug instead of actually a unused parameter? i think it should return and should also be in general fix

false // return
};

private _detectorType = param [1, currentWeapon _unit, [""]];
Copy link
Contributor

Choose a reason for hiding this comment

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

param still used here

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

Labels

ignore-changelog Release Notes: Excluded kind/cleanup Release Notes: **CHANGED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants