From eb82bf360bb6a461d41293b094feaf0a6caa7ba4 Mon Sep 17 00:00:00 2001 From: jerrrrycho Date: Sat, 10 Oct 2026 16:22:01 +0200 Subject: [PATCH] refactor(sonar): add default cases to switch statements (cpp:S3562) --- McZapkie/Mover.cpp | 14 ++++++++++++++ scripting/ladderlogic.cpp | 9 +++++++++ vehicle/Train.cpp | 2 ++ 3 files changed, 25 insertions(+) diff --git a/McZapkie/Mover.cpp b/McZapkie/Mover.cpp index 9cfa7e7d..0241380d 100644 --- a/McZapkie/Mover.cpp +++ b/McZapkie/Mover.cpp @@ -1996,6 +1996,8 @@ double TMoverParameters::ShowEngineRotation(int VehN) const if (TestFlag(Couplers[b].CouplingFlag, coupling::control) && Couplers[b].Connected->Power > 0.01 && TestFlag(Couplers[b].Connected->Couplers[b].CouplingFlag, coupling::control) && Couplers[b].Connected->Couplers[b].Connected->Power > 0.01) return fabs(Couplers[b].Connected->Couplers[b].Connected->enrot); break; + default: + break; } return 0.0; }; @@ -2484,6 +2486,8 @@ bool TMoverParameters::IncMainCtrl(int CtrlSpeed) OK = AddPulseForce(CtrlSpeed); break; } + default: + break; } // switch EngineType of } else @@ -2618,6 +2622,8 @@ bool TMoverParameters::DecMainCtrl(int CtrlSpeed) } break; } + default: + break; } // switch EngineType } } @@ -6251,6 +6257,8 @@ double TMoverParameters::TractionForce(double dt) SpeedCtrlValue = Vmax; SpeedCtrlTimer = 10; break; + default: + break; } } else if (ScndCtrlPosNo > 1) @@ -7321,6 +7329,8 @@ void TMoverParameters::CheckEIMIC(double dt) case 7: // TMax eimic += std::clamp(1.0 - eimic, 0.0, dt * 0.14); // dodawaj do 1, max break; + default: + break; } if (MainCtrlPos >= 3 && eimic < 0) eimic = 0; @@ -7350,6 +7360,8 @@ void TMoverParameters::CheckEIMIC(double dt) if (eimic < 0) eimic = 0; break; + default: + break; } } if (MainCtrlActualPos == MainCtrlPos) @@ -9573,6 +9585,8 @@ void TMoverParameters::BrakeSubsystemDecode() case TBrakeValve::EStED: BrakeSubsystem = TBrakeSubSystem::ss_LSt; break; + default: + break; } } diff --git a/scripting/ladderlogic.cpp b/scripting/ladderlogic.cpp index 23dbfb7e..362b48af 100644 --- a/scripting/ladderlogic.cpp +++ b/scripting/ladderlogic.cpp @@ -300,6 +300,8 @@ basic_controller::run() -> int { } break; } + default: + break; } // accumulator was published at least once, next ld(i) operation will start a new rung m_popstack = true; @@ -332,6 +334,8 @@ basic_controller::run() -> int { */ break; } + default: + break; } // accumulator was published at least once, next ld(i) operation will start a new rung m_popstack = true; @@ -354,11 +358,16 @@ basic_controller::run() -> int { std::get(target.data).count_value = 0; break; } + default: + break; } // accumulator was published at least once, next ld(i) operation will start a new rung m_popstack = true; break; } + default: { + break; + } } ++programline; } diff --git a/vehicle/Train.cpp b/vehicle/Train.cpp index 24834b78..09fdb0a1 100644 --- a/vehicle/Train.cpp +++ b/vehicle/Train.cpp @@ -10386,6 +10386,8 @@ bool TTrain::InitializeCab(int NewCabNo, std::string const &asFileName) case 0: cabindex = 0; break; + default: + break; } iCabn = cabindex;