Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #260, Remove registration of empty EVS filters #261

Merged
merged 1 commit into from
Jun 16, 2022

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Jun 14, 2022

Checklist (Please check before submitting)

Describe the contribution

Testing performed
CI

Expected behavior changes
Eliminates CFE_EVS_Registration filter truncation message and no longer registers empty filters

System(s) tested on
CI

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Jun 14, 2022
@skliper skliper added this to the Draco milestone Jun 14, 2022
@astrogeco astrogeco added CCB:Approved and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Jun 16, 2022
@astrogeco
Copy link
Contributor

astrogeco commented Jun 16, 2022

CCB:2022-06-15 APPROVED

@astrogeco astrogeco merged commit 593d61a into nasa:main Jun 16, 2022
@skliper skliper deleted the fix260-rm_empty_filters branch September 19, 2022 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Registering 100+ event filters w/ 0x0000 values serves no purpose (and overflows a typical filter buffer)
2 participants