FS CFE_FS_ParseInputFileNameEx missing branch coverage (defensive condition check)

[skliper]

Is your feature request related to a problem? Please describe.
Non-independent conditions checked in CFE_FS_ParseInputFileNameEx:

cFE/modules/fs/fsw/src/cfe_fs_api.c

Line 441 in 5e41330

while (InputPtr != NULL && Component < END_COMPONENT)

Since when processing EXTENSION_COMPONENT (the case before END_COMPONENT) the InputPtr is set to NULL:

cFE/modules/fs/fsw/src/cfe_fs_api.c

Lines 547 to 558 in 5e41330

	case EXTENSION_COMPONENT:
	/* Intentional fall through to default case */
	default:
	/* Just consume the rest of input -
	* should already be pointing to correct data */
	ComponentTerm = 0;
	ComponentLen = InputLen;
	ComponentPtr = InputPtr;
	InputPtr = NULL; /* no more input */
	InputLen = 0;
	break;

Describe the solution you'd like
Trade the usefulness of testing for an unreachable condition. It's "defensive" but only reachable by changing the implementation, although it can't cause harm.

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC

[avan989]

With the current implementation, the loop will exit before the Component reaches "END_COMPONENT" making this branch unreachable. Recommend closing tickets (and if needed open a new ticket to change implementation). @dmknutsen @dzbaker