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

Document how to change CFE_TIME_Print() timestamp format #2490

Open
CDKnightNASA opened this issue Jan 4, 2024 · 0 comments
Open

Document how to change CFE_TIME_Print() timestamp format #2490

CDKnightNASA opened this issue Jan 4, 2024 · 0 comments

Comments

@CDKnightNASA
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Some missions may want to use a different timestamp format for event messages on the console (such as "MET" or "DD HH:MM:SS" or whatnot.) A proposal to use strftime made the logic more complex and did have a measurable performance impact, so instead I suggest documenting how a mission would change the format and cautioning against using a strftime or sprintf formatting. I suggest the comment be added to the CFE_TIME_Print() definition in cfe_time.h.

Describe the solution you'd like
See description above. Comment in cfe_time.h.

Describe alternatives you've considered
Do nothing. Add comment to cfe_time_api.c. Add sample code in comments. Whatnot.

Additional context
See related, withdrawn PR #2370

Requester Info
[email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant