nasa / LC

The Core Flight System (cFS) Limit Checker (LC) application.
Apache License 2.0
30 stars 21 forks source link

Fix #77, Add break to default case of switch in LC_VerifyMsgLength #78

Closed thnkslprpt closed 1 year ago

thnkslprpt commented 1 year ago

Checklist

Describe the contribution

Testing performed GitHub CI actions (incl. Build + Run, Unit Tests etc.) all passing successfully.

Expected behavior changes Purely a style/guidelines issue for consistency and future maintenance. No change to logic (the default case would fall through to the bottom and exit the switch block even without a break statement). Guidelines and cFS consistency dictate that all switch cases (including default) should be terminated by an unconditional break statement.

Contributor Info Avi Weiss @thnkslprpt