Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Resolve cppcheck redundantAssignment failure #730

Closed
skliper opened this issue Jan 4, 2021 · 0 comments · Fixed by #731
Closed

Resolve cppcheck redundantAssignment failure #730

skliper opened this issue Jan 4, 2021 · 0 comments · Fixed by #731
Labels
continuous-integration enhancement
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Jan 4, 2021

Is your feature request related to a problem? Please describe.
Running -

cppcheck --force --inline-suppr --std=c99 --language=c --error-exitcode=1 --enable=warning,performance,portability,style --suppress=variableScope --inconclusive --quiet src/bsp src/os 2>cppcheck_flight_osal.txt

Results in -

[src/os/shared/src/osapi-idmap.c:753] -> [src/os/shared/src/osapi-idmap.c:765]: (style) Variable 'objtype->table_owner' is reassigned a value before the old one has been used.

Describe the solution you'd like
Resolve

Describe alternatives you've considered
None

Additional context
travis CI no longer running on main repo

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper mentioned this issue Jan 4, 2021
Merged
@skliper skliper changed the title Resolve cppcheck failure Resolve cppcheck redundantAssignment failure Jan 4, 2021
@astrogeco astrogeco added this to the 6.0.0 milestone Jan 5, 2021
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@CDKnightNASA
fix nasa#730 - adds null to FILEDEFs
c37197e
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@astrogeco
Merge pull request nasa#738 from CDKnightNASA/fix-730-tbl_filedef_nulls
dcb00a4 
Fx nasa#730, Adds null to FILEDEFs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
continuous-integration enhancement
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #730, Suppress invalid redundantAssignment warning from cppcheck skliper/osal
2 participants
@skliper @astrogeco