Should CFE_EVS_Register() be called with CFE_EVS_NO_FILTER or CFE_EVS_EventFilter_BINARY?

[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
Shouldn't SC be calling CFE_EVS_Register() with CFE_EVS_EventFilter_BINARY for the FilterScheme parameter? (rather than CFE_EVS_NO_FILTER)

They both end up as the integer zero, so the check here will treat them the same:
https://github.com/nasa/cFE/blob/7479d221692672d66946081f1cc5eb7c1d2acf9b/modules/evs/fsw/src/cfe_evs.c#L59-L68

A couple of other apps (CS and DS) actually just call it directly with 0 like so:
Result = CFE_EVS_Register(NULL, 0, 0);
...which also seems wrong (they should also just use CFE_EVS_EventFilter_BINARY directly).

Code snips

SC/fsw/src/sc_app.c

Line 172 in 487169b

Result = CFE_EVS_Register(NULL, 0, CFE_EVS_NO_FILTER);

Reporter Info
Avi Weiss @thnkslprpt