-
Notifications
You must be signed in to change notification settings - Fork 48
ogma-core: cFS template has tab characters #294
Copy link
Copy link
Closed
Labels
CR:Status:ClosedAdmin only: Change request that has been completedAdmin only: Change request that has been completedCR:Type:BugAdmin only: Change request pertaining to error detectedAdmin only: Change request pertaining to error detected
Milestone
Description
Description
The cFS template has tab characters, but shouldn't. Tab characters make it harder for alignment in code to remain consistent across developers, and are highly discouraged or forbidden by style guides, including Ogma's.
Type
- Bug: Code does not meet style guide.
Additional context
None.
Requester
- Ivan Perez.
Method to check presence of bug
Running the following grep command brings up a few occurrences of tabs in the cFS template:
$ grep -nHrP '\t' ogma-core/templates/copilot-cfs/
ogma-core/templates/copilot-cfs/fsw/src/copilot_cfs.c:147: "COPILOT: invalid command packet,MID = 0x%x", MsgId);
ogma-core/templates/copilot-cfs/fsw/platform_inc/copilot_cfs_msgids.h:15:#define COPILOT_CFS_CMD_MID 0x1882
ogma-core/templates/copilot-cfs/fsw/platform_inc/copilot_cfs_msgids.h:16:#define COPILOT_CFS_SEND_HK_MID 0x1883
ogma-core/templates/copilot-cfs/fsw/platform_inc/copilot_cfs_msgids.h:17:#define COPILOT_CFS_HK_TLM_MID 0x0883
Expected result
The cFS template contains no tabs. Running the grep command above shows no results.
Desired result
The cFS template contains no tabs. Running the grep command above shows no results.
Proposed solution
Remove tabs from cFS templates. Align code appropriately.
Further notes
None.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
CR:Status:ClosedAdmin only: Change request that has been completedAdmin only: Change request that has been completedCR:Type:BugAdmin only: Change request pertaining to error detectedAdmin only: Change request pertaining to error detected