CreatePipeErrorCounter only incremented for CFE_SB_CR_PIPE_BAD_ARG_EID or in CFE_SB_DeletePipeFull

[skliper]

Is your feature request related to a problem? Please describe.
CreatePipeErrorCounter logic doesn't seem to make all that much sense since it's only incremented for one error case, and also incremented as part of deleting a pipe.

Note the trivial switch below is a static analysis warning, might be more appropriate to count warnings for more cases:

cFE/fsw/cfe-core/src/sb/cfe_sb_api.c

Lines 227 to 235 in c708d64

	switch(PendingEventId)
	{
	case CFE_SB_CR_PIPE_BAD_ARG_EID:
	++CFE_SB_Global.HKTlmMsg.Payload.CreatePipeErrorCounter;
	break;
	default:
	/* no counter */
	break;
	}

Describe the solution you'd like
Trace design to requirements, confirm current implementation makes sense.

Describe alternatives you've considered
None

Additional context
Uncovered due to static analysis warning, any change will be easier after #972

Requester Info
Jacob Hageman - NASA/GSFC