-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add break; for switch default case in LC_VerifyMsgLength() #77
Labels
Milestone
Comments
thnkslprpt
added a commit
to thnkslprpt/LC
that referenced
this issue
Mar 13, 2023
2 tasks
thnkslprpt
added a commit
to thnkslprpt/LC
that referenced
this issue
Mar 13, 2023
thnkslprpt
added a commit
to thnkslprpt/LC
that referenced
this issue
Mar 13, 2023
thnkslprpt
added a commit
to thnkslprpt/LC
that referenced
this issue
Mar 13, 2023
thnkslprpt
added a commit
to thnkslprpt/LC
that referenced
this issue
Mar 13, 2023
thnkslprpt
added a commit
to thnkslprpt/LC
that referenced
this issue
Mar 14, 2023
thnkslprpt
added a commit
to thnkslprpt/LC
that referenced
this issue
Apr 17, 2023
dzbaker
added a commit
that referenced
this issue
Apr 27, 2023
…-switch Fix #77, Add break to default case of switch in LC_VerifyMsgLength
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Checklist
Describe the bug
default
case of theswitch
block inLC_VerifyMsgLength()
is missing abreak;
.Purely a style/guidelines issue for consistency and future maintenance.
Code snips
LC/fsw/src/lc_utils.c
Lines 83 to 92 in 2f177ae
Expected behavior
All switch cases (including
default
) should be terminated by an unconditionalbreak
statement.Reporter Info
Avi Weiss @thnkslprpt
The text was updated successfully, but these errors were encountered: