nasa / LC

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

Add break; for switch default case in LC_VerifyMsgLength() #77

Closed thnkslprpt closed 1 year ago

thnkslprpt commented 1 year ago

Checklist

Describe the bug default case of the switch block in LC_VerifyMsgLength() is missing a break;. Purely a style/guidelines issue for consistency and future maintenance.

Code snips https://github.com/nasa/LC/blob/2f177ae83a24445d6ab6997682a2ffa71dacbd31/fsw/src/lc_utils.c#L83-L92

Expected behavior All switch cases (including default) should be terminated by an unconditional break statement.

Reporter Info Avi Weiss @thnkslprpt