Fix #1456, Rename CFE_SB_RemovePipeCallback_t to ...backArgs_t

[thnkslprpt]

Checklist

• I reviewed the Contributing Guide.
• I signed and emailed the appropriate Contributor License Agreement to [email protected] and copied [email protected].

Describe the contribution

• Fixes CFE_SB_RemovePipeCallback_t typedef implies a callback, but it's callback arguments #1456
  • Renames CFE_SB_RemovePipeCallback_t to CFE_SB_RemovePipeCallbackArgs_t, which better matches what the struct represents.
    • Only used locally in 2 functions.

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

Expected behavior changes
No impact on behavior.

Contributor Info
Avi Weiss @thnkslprpt

[jphickey]

Looks good, but also something we should consider documenting in https://github.com/nasa/cFE/blob/main/docs/cFS_IdentifierNamingConvention.md as well (given that names are almost always subjective, if we are going to be doing PRs to fix names then we should have something to fall back on that says what is "right")

[thnkslprpt]

Looks good, but also something we should consider documenting in https://github.com/nasa/cFE/blob/main/docs/cFS_IdentifierNamingConvention.md as well (given that names are almost always subjective, if we are going to be doing PRs to fix names then we should have something to fall back on that says what is "right")

Good point - I forgot to update that doc. Will do so today.