From a45ca270369c2b4c133534a36695724a975723d0 Mon Sep 17 00:00:00 2001 From: Avi Date: Wed, 28 Sep 2022 15:03:53 +1000 Subject: [PATCH] Fix #48, Remove 'return;' from last line of void functions. --- fsw/src/cs_app.c | 5 ----- fsw/src/cs_app_cmds.c | 6 ------ fsw/src/cs_cmds.c | 17 ----------------- fsw/src/cs_compute.c | 7 ------- fsw/src/cs_eeprom_cmds.c | 7 ------- fsw/src/cs_events.h | 6 +++--- fsw/src/cs_memory_cmds.c | 8 -------- fsw/src/cs_table_cmds.c | 6 ------ fsw/src/cs_table_processing.c | 3 --- fsw/src/cs_utils.c | 17 ----------------- 10 files changed, 3 insertions(+), 79 deletions(-) diff --git a/fsw/src/cs_app.c b/fsw/src/cs_app.c index bb957e5..3632e14 100644 --- a/fsw/src/cs_app.c +++ b/fsw/src/cs_app.c @@ -566,8 +566,6 @@ void CS_HousekeepingCmd(const CS_NoArgsCmd_t *CmdPtr) CFE_SB_TimeStampMsg(&CS_AppData.HkPacket.TlmHeader.Msg); CFE_SB_TransmitMsg(&CS_AppData.HkPacket.TlmHeader.Msg, true); } - - return; } /* End of CS_HousekeepingCmd () */ #if (CS_PRESERVE_STATES_ON_PROCESSOR_RESET == true) @@ -710,9 +708,6 @@ void CS_UpdateCDS(void) CS_AppData.DataStoreHandle = CFE_ES_CDS_BAD_HANDLE; } } - - return; - } /* End of CS_UpdateCDS() */ #endif /* #if (CS_PRESERVE_STATES_ON_PROCESSOR_RESET == true ) */ diff --git a/fsw/src/cs_app_cmds.c b/fsw/src/cs_app_cmds.c index ac4e272..d1f025b 100644 --- a/fsw/src/cs_app_cmds.c +++ b/fsw/src/cs_app_cmds.c @@ -67,7 +67,6 @@ void CS_DisableAppCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } } - return; } /* End of CS_DisableAppCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -97,7 +96,6 @@ void CS_EnableAppCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } } - return; } /* End of CS_EnableAppCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -142,7 +140,6 @@ void CS_ReportBaselineAppCmd(const CS_AppNameCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* End of CS_ReportBaselineAppCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -212,7 +209,6 @@ void CS_RecomputeBaselineAppCmd(const CS_AppNameCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* end CS_RecomputeBaselineAppCmd */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -270,7 +266,6 @@ void CS_DisableNameAppCmd(const CS_AppNameCmd_t *CmdPtr) } } /* end InProgress if */ } - return; } /* End of CS_DisableNameAppCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -324,7 +319,6 @@ void CS_EnableNameAppCmd(const CS_AppNameCmd_t *CmdPtr) } } /* end InProgress if */ } - return; } /* End of CS_EnableNameAppCmd () */ /************************/ diff --git a/fsw/src/cs_cmds.c b/fsw/src/cs_cmds.c index 69b0493..1b40f0c 100644 --- a/fsw/src/cs_cmds.c +++ b/fsw/src/cs_cmds.c @@ -59,7 +59,6 @@ void CS_NoopCmd(const CS_NoArgsCmd_t *CmdPtr) CFE_EVS_SendEvent(CS_NOOP_INF_EID, CFE_EVS_EventType_INFORMATION, "No-op command. Version %d.%d.%d.%d", CS_MAJOR_VERSION, CS_MINOR_VERSION, CS_REVISION, CS_MISSION_REV); } - return; } /* End of CS_NoopCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -88,7 +87,6 @@ void CS_ResetCmd(const CS_NoArgsCmd_t *CmdPtr) CFE_EVS_SendEvent(CS_RESET_DBG_EID, CFE_EVS_EventType_DEBUG, "Reset Counters command recieved"); } - return; } /* End of CS_ResetCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -198,7 +196,6 @@ void CS_BackgroundCheckCycle(const CS_NoArgsCmd_t *CmdPtr) /* CS is disabled, Application-wide */ } } - return; } /* End of CS_BackgroundCheckCycle () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -230,7 +227,6 @@ void CS_DisableAllCSCmd(const CS_NoArgsCmd_t *CmdPtr) CFE_EVS_SendEvent(CS_DISABLE_ALL_INF_EID, CFE_EVS_EventType_INFORMATION, "Background Checksumming Disabled"); } - return; } /* End of CS_DisableAllCSCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -252,7 +248,6 @@ void CS_EnableAllCSCmd(const CS_NoArgsCmd_t *CmdPtr) CFE_EVS_SendEvent(CS_ENABLE_ALL_INF_EID, CFE_EVS_EventType_INFORMATION, "Background Checksumming Enabled"); } - return; } /* End of CS_EnableAllCSCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -280,7 +275,6 @@ void CS_DisableCfeCoreCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } - return; } /* End of CS_DisableCfeCoreCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -307,9 +301,6 @@ void CS_EnableCfeCoreCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } - - return; - } /* End of CS_EnableCfeCoreCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -337,7 +328,6 @@ void CS_DisableOSCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } - return; } /* End of CS_DisableOSCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -364,7 +354,6 @@ void CS_EnableOSCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } - return; } /* End of CS_OSEnableCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -393,7 +382,6 @@ void CS_ReportBaselineCfeCoreCmd(const CS_NoArgsCmd_t *CmdPtr) } CS_AppData.HkPacket.CmdCounter++; } - return; } /* End of CS_ReportBaselineCfeCoreCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -422,7 +410,6 @@ void CS_ReportBaselineOSCmd(const CS_NoArgsCmd_t *CmdPtr) } CS_AppData.HkPacket.CmdCounter++; } - return; } /* End of CS_ReportBaselineOSCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -477,7 +464,6 @@ void CS_RecomputeBaselineCfeCoreCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* end CS_RecomputeBaselineCfeCoreCmd */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -530,7 +516,6 @@ void CS_RecomputeBaselineOSCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* end CS_RecomputeBaselineOSCmd */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -613,7 +598,6 @@ void CS_OneShotCmd(const CS_OneShotCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* end CS_OneShotCmd */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -659,7 +643,6 @@ void CS_CancelOneShotCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* end CS_CancelOneShotCmd */ /************************/ diff --git a/fsw/src/cs_compute.c b/fsw/src/cs_compute.c index 5b34a10..55ef0bd 100644 --- a/fsw/src/cs_compute.c +++ b/fsw/src/cs_compute.c @@ -564,8 +564,6 @@ void CS_RecomputeEepromMemoryChildTask(void) CS_AppData.HkPacket.RecomputeInProgress = false; CFE_ES_ExitChildTask(); - - return; } /* end CS_RecomputeEepromMemoryChildTask */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -672,8 +670,6 @@ void CS_RecomputeAppChildTask(void) CS_AppData.HkPacket.RecomputeInProgress = false; CFE_ES_ExitChildTask(); - - return; } /* end CS_RecomputeAppChildTask */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -780,8 +776,6 @@ void CS_RecomputeTablesChildTask(void) CS_AppData.HkPacket.RecomputeInProgress = false; CFE_ES_ExitChildTask(); - - return; } /* end CS_RecomputeTablesChildTask */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -832,7 +826,6 @@ void CS_OneShotChildTask(void) CS_AppData.ChildTaskID = CFE_ES_TASKID_UNDEFINED; CFE_ES_ExitChildTask(); - return; } /* end CS_OneShotChildTask */ /************************/ diff --git a/fsw/src/cs_eeprom_cmds.c b/fsw/src/cs_eeprom_cmds.c index 30525fd..b7f3e96 100644 --- a/fsw/src/cs_eeprom_cmds.c +++ b/fsw/src/cs_eeprom_cmds.c @@ -71,7 +71,6 @@ void CS_DisableEepromCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } } - return; } /* End of CS_DisableEepromCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -103,7 +102,6 @@ void CS_EnableEepromCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } } - return; } /* End of CS_EnableEepromCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -162,7 +160,6 @@ void CS_ReportBaselineEntryIDEepromCmd(const CS_EntryCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* End of CS_ReportBaselineEntryIDCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -247,7 +244,6 @@ void CS_RecomputeBaselineEepromCmd(const CS_EntryCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* end CS_RecomputeBaselineEepromCmd */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -316,7 +312,6 @@ void CS_EnableEntryIDEepromCmd(const CS_EntryCmd_t *CmdPtr) } } /* end InProgress if */ } - return; } /* End of CS_EnableCSEntryIDEepromCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -387,7 +382,6 @@ void CS_DisableEntryIDEepromCmd(const CS_EntryCmd_t *CmdPtr) } } /* end InProgress if */ } - return; } /* End of CS_DisableCSEntryIDEepromCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -432,7 +426,6 @@ void CS_GetEntryIDEepromCmd(const CS_GetEntryIDCmd_t *CmdPtr) } CS_AppData.HkPacket.CmdCounter++; } - return; } /* End of CS_GetEntryIDEepromCmd () */ /************************/ diff --git a/fsw/src/cs_events.h b/fsw/src/cs_events.h index d325a69..afbeb37 100644 --- a/fsw/src/cs_events.h +++ b/fsw/src/cs_events.h @@ -1257,7 +1257,7 @@ * \par Cause: * * This event message is issued when a recompute entry for Eeprom or Memory - * has finished sucesessfully. + * has finished succesessfully. */ #define CS_RECOMPUTE_FINISH_EEPROM_MEMORY_INF_EID 95 @@ -1293,7 +1293,7 @@ * \par Cause: * * This event message is issued when a recompute entry for a Table - * has finished sucesessfully. + * has finished succesessfully. * * For the CS Table Definitions Table only, the checksum * value will be incorrect. This is because all entries in this @@ -1312,7 +1312,7 @@ * \par Cause: * * This event message is issued when a recompute entry for an app - * has finished sucesessfully. + * has finished succesessfully. */ #define CS_RECOMPUTE_FINISH_APP_INF_EID 99 diff --git a/fsw/src/cs_memory_cmds.c b/fsw/src/cs_memory_cmds.c index e87cbf1..8c19b2e 100644 --- a/fsw/src/cs_memory_cmds.c +++ b/fsw/src/cs_memory_cmds.c @@ -70,7 +70,6 @@ void CS_DisableMemoryCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } } - return; } /* End of CS_DisableMemoryCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -102,7 +101,6 @@ void CS_EnableMemoryCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } } - return; } /* End of CS_EnableMemoryCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -160,7 +158,6 @@ void CS_ReportBaselineEntryIDMemoryCmd(const CS_EntryCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* End of CS_ReportBaselineEntryIDCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -245,7 +242,6 @@ void CS_RecomputeBaselineMemoryCmd(const CS_EntryCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* end CS_RecomputeBaselineMemoryCmd */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -311,7 +307,6 @@ void CS_EnableEntryIDMemoryCmd(const CS_EntryCmd_t *CmdPtr) } } /* end InProgress if */ } - return; } /* End of CS_EnableCSEntryIDMemoryCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -381,7 +376,6 @@ void CS_DisableEntryIDMemoryCmd(const CS_EntryCmd_t *CmdPtr) } } /* end InProgress if */ } - return; } /* End of CS_DisableCSEntryIDMemoryCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -425,8 +419,6 @@ void CS_GetEntryIDMemoryCmd(const CS_GetEntryIDCmd_t *CmdPtr) } CS_AppData.HkPacket.CmdCounter++; } - return; - } /* End of CS_GetEntryIDMemoryCmd () */ /************************/ diff --git a/fsw/src/cs_table_cmds.c b/fsw/src/cs_table_cmds.c index 2ddf10e..9cb9ae0 100644 --- a/fsw/src/cs_table_cmds.c +++ b/fsw/src/cs_table_cmds.c @@ -67,7 +67,6 @@ void CS_DisableTablesCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } } - return; } /* End of CS_DisableTablesCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -98,7 +97,6 @@ void CS_EnableTablesCmd(const CS_NoArgsCmd_t *CmdPtr) CS_AppData.HkPacket.CmdCounter++; } } - return; } /* End of CS_EnableTablesCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -143,7 +141,6 @@ void CS_ReportBaselineTablesCmd(const CS_TableNameCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* End of CS_ReportBaselineTablesCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -214,7 +211,6 @@ void CS_RecomputeBaselineTablesCmd(const CS_TableNameCmd_t *CmdPtr) CS_AppData.HkPacket.CmdErrCounter++; } } - return; } /* end CS_RecomputeBaselineTablesCmd */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -271,7 +267,6 @@ void CS_DisableNameTablesCmd(const CS_TableNameCmd_t *CmdPtr) } } /* end InProgress if */ } - return; } /* End of CS_DisableNameTablesCmd () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -325,7 +320,6 @@ void CS_EnableNameTablesCmd(const CS_TableNameCmd_t *CmdPtr) } } /* end InProgress if */ } - return; } /* End of CS_EnableNameTablesCmd () */ /************************/ diff --git a/fsw/src/cs_table_processing.c b/fsw/src/cs_table_processing.c index ddaf2ec..4891ec6 100644 --- a/fsw/src/cs_table_processing.c +++ b/fsw/src/cs_table_processing.c @@ -528,7 +528,6 @@ void CS_ProcessNewEepromMemoryDefinitionTable(const CS_Def_EepromMemory_Table_En CFE_EVS_SendEvent(CS_PROCESS_EEPROM_MEMORY_NO_ENTRIES_INF_EID, CFE_EVS_EventType_INFORMATION, "CS %s Table: No valid entries in the table", TableType); } - return; } /* end of CS_ProcessNewEepromMemoryDefinitionTable () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -721,7 +720,6 @@ void CS_ProcessNewTablesDefinitionTable(const CS_Def_Tables_Table_Entry_t *Defin CFE_EVS_SendEvent(CS_PROCESS_TABLES_NO_ENTRIES_INF_EID, CFE_EVS_EventType_INFORMATION, "CS Tables Table: No valid entries in the table"); } - return; } /* end of CS_ProcessNewTablesDefinitionTable () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -794,7 +792,6 @@ void CS_ProcessNewAppDefinitionTable(const CS_Def_App_Table_Entry_t *DefinitionT CFE_EVS_SendEvent(CS_PROCESS_APP_NO_ENTRIES_INF_EID, CFE_EVS_EventType_INFORMATION, "CS Apps Table: No valid entries in the table"); } - return; } /* end of CS_ProcessNewAppsDefinitionTable () */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ diff --git a/fsw/src/cs_utils.c b/fsw/src/cs_utils.c index c51e857..d6f23d7 100644 --- a/fsw/src/cs_utils.c +++ b/fsw/src/cs_utils.c @@ -53,7 +53,6 @@ void CS_ZeroEepromTempValues(void) CS_AppData.ResEepromTblPtr[Loop].TempChecksumValue = 0; CS_AppData.ResEepromTblPtr[Loop].ByteOffset = 0; } - return; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -70,8 +69,6 @@ void CS_ZeroMemoryTempValues(void) CS_AppData.ResMemoryTblPtr[Loop].TempChecksumValue = 0; CS_AppData.ResMemoryTblPtr[Loop].ByteOffset = 0; } - - return; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -88,8 +85,6 @@ void CS_ZeroTablesTempValues(void) CS_AppData.ResTablesTblPtr[Loop].TempChecksumValue = 0; CS_AppData.ResTablesTblPtr[Loop].ByteOffset = 0; } - - return; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -106,8 +101,6 @@ void CS_ZeroAppTempValues(void) CS_AppData.ResAppTblPtr[Loop].TempChecksumValue = 0; CS_AppData.ResAppTblPtr[Loop].ByteOffset = 0; } - - return; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ /* */ @@ -118,8 +111,6 @@ void CS_ZeroCfeCoreTempValues(void) { CS_AppData.CfeCoreCodeSeg.TempChecksumValue = 0; CS_AppData.CfeCoreCodeSeg.ByteOffset = 0; - - return; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -131,8 +122,6 @@ void CS_ZeroOSTempValues(void) { CS_AppData.OSCodeSeg.TempChecksumValue = 0; CS_AppData.OSCodeSeg.ByteOffset = 0; - - return; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ /* */ @@ -168,8 +157,6 @@ void CS_InitializeDefaultTables(void) CS_AppData.DefaultTablesDefTable[Loop].State = CS_STATE_EMPTY; CS_AppData.DefaultTablesDefTable[Loop].Name[0] = '\0'; } - - return; } /* end CS_InitializeDefaultTables */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -192,7 +179,6 @@ void CS_GoToNextTable(void) } CS_AppData.HkPacket.CurrentEntryInTable = 0; - return; } /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */ @@ -929,9 +915,6 @@ void CS_ResetTablesTblResultEntry(CS_Res_Tables_Table_Entry_t *TablesTblResultEn TablesTblResultEntry->TempChecksumValue = 0; TablesTblResultEntry->ComputedYet = false; } - - return; - } /* end CS_ResetTablesTblResultEntry */ /* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */