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.

Sign up for GitHub

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

Jump to bottom

Remove ES_APP_DEBUG ifdef and related code #639

Closed
skliper opened this issue Apr 24, 2020 · 1 comment · Fixed by #646 or #692
Closed

Remove ES_APP_DEBUG ifdef and related code #639

skliper opened this issue Apr 24, 2020 · 1 comment · Fixed by #646 or #692
Assignees
@skliper
Labels
enhancement
Milestone
Bootes

Comments

@skliper
Copy link
Contributor

skliper commented Apr 24, 2020

Is your feature request related to a problem? Please describe.
Debug code exists in flight code, uses printf, ifdefed out with ES_APP_DEBUG. Undocumented option.

Describe the solution you'd like
Remove.

Describe alternatives you've considered
N/A

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 6.8.0 milestone Apr 24, 2020
@skliper
Copy link
Contributor Author

skliper commented Apr 24, 2020

Somewhat duplicate (although more complex) logic of CFE_ES_ListResources.

skliper added a commit to skliper/cFE that referenced this issue Apr 28, 2020
@skliper
Fix nasa#639, Remove ES_APP_DEBUG functionality
d264cb3 
Deletes now unused CFE_ES_CountObjectCallback
and CFE_ES_ListResourcesDebug.
skliper added a commit to skliper/cFS that referenced this issue Apr 28, 2020
@skliper
Test nasa/cFE#639
a435c52
@skliper skliper mentioned this issue Apr 28, 2020
Merged
@skliper skliper self-assigned this Apr 28, 2020
skliper added a commit to skliper/cFE that referenced this issue Apr 28, 2020
@skliper
Fix nasa#639, Remove ES_APP_DEBUG functionality
8de3ab9 
Deletes now unused CFE_ES_CountObjectCallback
and CFE_ES_ListResourcesDebug.
skliper added a commit to skliper/cFE that referenced this issue May 8, 2020
@skliper
Fix nasa#639, Remove ES_APP_DEBUG functionality
9cb0b5d 
Deletes now unused CFE_ES_CountObjectCallback
and CFE_ES_ListResourcesDebug.
astrogeco added a commit that referenced this issue May 8, 2020
@astrogeco
Merge pull request #646 from skliper/fix639-es-app-debug
10e7f81 
Fix #639, Remove ES_APP_DEBUG functionality
skliper added a commit to skliper/cFE that referenced this issue May 11, 2020
@skliper
Merge pull request nasa#646 from skliper/fix639-es-app-debug
99f670b 
Fix nasa#639, Remove ES_APP_DEBUG functionality
@astrogeco astrogeco mentioned this issue May 13, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
enhancement
Projects
None yet
Milestone
Bootes
1 participant
@skliper