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

Use CFE_TIME_SysTime_t instead of individual Seconds/subseconds #1497

Open
skliper opened this issue May 11, 2021 · 0 comments · May be fixed by #2246
Open

Use CFE_TIME_SysTime_t instead of individual Seconds/subseconds #1497

skliper opened this issue May 11, 2021 · 0 comments · May be fixed by #2246

Comments

@skliper
Copy link
Contributor

skliper commented May 11, 2021

Is your feature request related to a problem? Please describe.
Inconsistent use/definition of time structure vs individual sec/subsecs, benefits from common structure:

uint32 FileCreateTimeSecs; /**< \brief File creation time from last file loaded into table */
uint32 FileCreateTimeSubSecs; /**< \brief File creation time from last file loaded into table */
uint32 Crc; /**< \brief Most recently calculated CRC by TBL services on table contents */
CFE_TIME_SysTime_t TimeOfLastUpdate; /**< \brief Time when Table was last updated */

Describe the solution you'd like
Use CFE_TIME_SysTime_t

Describe alternatives you've considered
None

Additional context
Code review

Requester Info
Jacob Hageman - NASA/GSFC

@skliper skliper modified the milestone: 7.0.0 May 11, 2021
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Feb 16, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Apr 2, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Apr 3, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Apr 3, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Apr 3, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue May 5, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue May 5, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue May 5, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Oct 26, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Oct 26, 2023
thnkslprpt added a commit to thnkslprpt/cFE that referenced this issue Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant