CS_AttemptTableReshare can be simplified

[thnkslprpt]

Checklist

• I reviewed the Contributing Guide.
• I performed a cursory search to see if the bug report is relevant, not redundant, nor in conflict with other tickets.

Describe the bug
The CS_AttemptTableReshare() function uses 3 'Result' variables for control flow. It seems like they can all be combined into a single variable.

Code snips

CS/fsw/src/cs_utils.c

Lines 1004 to 1038 in a249bcf

	int32 CS_AttemptTableReshare(CS_Res_Tables_Table_Entry_t *ResultsEntry, CFE_TBL_Handle_t *LocalTblHandle,
	CFE_TBL_Info_t *TblInfo, cpuaddr *LocalAddress, int32 *ResultGetInfo)
	{
	int32 Result = CS_SUCCESS;
	int32 ResultShare = 0;
	int32 ResultGetAddress = 0;
	/* Maybe the table came back, try and reshare it */
	ResultShare = CFE_TBL_Share(LocalTblHandle, ResultsEntry->Name);
	if (ResultShare == CFE_SUCCESS)
	{
	ResultsEntry->TblHandle = *LocalTblHandle;
	*ResultGetInfo = CFE_TBL_GetInfo(TblInfo, ResultsEntry->Name);
	/* need to try to get the address again */
	ResultGetAddress = CFE_TBL_GetAddress((void *)LocalAddress, *LocalTblHandle);
	Result = ResultGetAddress;
	/* if the table was never loaded, release the address to prevent the table from being
	locked by CS, which would prevent the owner app from updating it*/
	if (ResultGetAddress == CFE_TBL_ERR_NEVER_LOADED)
	{
	CFE_TBL_ReleaseAddress(*LocalTblHandle);
	}
	}
	else /* table was not there on the new share */
	{
	Result = ResultShare;
	}
	return Result;
	}

Expected behavior
Simplify function to improve clarity and ease future maintenance.

Reporter Info
Avi Weiss @thnkslprpt