New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix #10, Rename `MM_ResetHk` to `MM_ResetHkData` + memset global data at initialization #91

Open

thnkslprpt wants to merge 1 commit into nasa:main from thnkslprpt:fix-10-rename-MM_ResetHk-to-MM_ResetHkData

Contributor

thnkslprpt commented Jan 17, 2024

Checklist

I reviewed the Contributing Guide.
I signed and emailed the appropriate Contributor License Agreement to GSFC-SoftwareRelease@mail.nasa.gov and copied cfs-program@lists.nasa.gov.

Describe the contribution

Fixes Function names MM_ResetHk and MM_WriteMemWID are misleading #10
- MM_ResetHk renamed to MM_ResetHkData to improve clarity
- Minor issues tagging along with this PR:
  - Zero out the entire global data structure during initialization (rather than just the command/error counters)
  - Separate the 2 separate words SYM and TBL in MM_SYMTBL_TO_FILE_CC as per the naming conventions
  - Various typo fixes

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

Expected behavior changes
No change.

Contributor Info
Avi Weiss @thnkslprpt


          Fix nasa#10, Rename MM_ResetHk to MM_ResetHkData to improve clarity

c80797a

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet