Skip to content

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

Merged
astrogeco merged 1 commit intonasa:mainfrom
skliper:fix260-rm_empty_filters
Jun 16, 2022
Merged

Fix #260, Remove registration of empty EVS filters#261
astrogeco merged 1 commit intonasa:mainfrom
skliper:fix260-rm_empty_filters

Conversation

@skliper
Copy link
Copy Markdown

@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
Copy Markdown
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