You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
CS also has a few command code macros that are not consistent with the enumeration identifiers or the command function name (all are cases of 2 words being combined into 1).
Code snips CS_OneShotCmd() CS_OneShotCmd_t CS_ONESHOT_CC (should be CS_ONE_SHOT_CC)
CS_EnableCfeCoreCmd() CS_ENABLE_CFECORE_CC (should be CS_ENABLE_CFE_CORE_CC)
...and a couple more.
Expected behavior
Identifiers should conform to the cFS/cFE naming convention with regard to consistent use of underscores to make mnemonics clear and readable.
Checklist
Describe the bug
Code snips
CS_OneShotCmd()
CS_OneShotCmd_t
CS_ONESHOT_CC
(should beCS_ONE_SHOT_CC
)CS_EnableCfeCoreCmd()
CS_ENABLE_CFECORE_CC
(should beCS_ENABLE_CFE_CORE_CC
)...and a couple more.
Expected behavior
Identifiers should conform to the cFS/cFE naming convention with regard to consistent use of underscores to make mnemonics clear and readable.
Reporter Info
Avi Weiss @thnkslprpt
The text was updated successfully, but these errors were encountered: