Implement single pattern for command handler returns

[skliper]

Is your feature request related to a problem? Please describe.

• ES ignores command handler returns, just returns CFE_SUCCSS (increments counters within handler)
• EVS captures the return and increments appropriate counter as part of the main command processing routine
• TBL defines CFE_TBL_CmdProcRet_t but doesn't use it consistently, example:
  

  cFE/modules/tbl/fsw/src/cfe_tbl_task_cmds.h

  Line 311 in 84ba9a9

  int32 CFE_TBL_AbortLoadCmd(const CFE_TBL_AbortLoadCmd_t *data);

  
  

  cFE/modules/tbl/fsw/src/cfe_tbl_task_cmds.h

  Lines 337 to 338 in 84ba9a9

  	extern CFE_TBL_CmdProcRet_t CFE_TBL_DumpToFile(const char *DumpFilename, const char *TableName,
  	const void *DumpDataAddr, size_t TblSizeInBytes);

Describe the solution you'd like
Utilize a consistent pattern. Suggest using CFE_Status_t and defined error types. This allows command handlers to use APIs and not need to convert error codes.

Describe alternatives you've considered
None

Additional context
Code review, also related to issue #921 to consistently use CFE_Status_t

Requester Info
Jacob Hageman - NASA/GSFC