Added an additional conversion function #584
No reviewers
Labels
No Label
API Change
Breaking API Change
bug
build
cosmetics
Documentation
duplicate
feature
help wanted
hotfix
invalid
question
Refactor
Tests
wontfix
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Blocks
#593 CCSDS Time CUC Tests
fsfw/fsfw
Reference: fsfw/fsfw#584
Loading…
Reference in New Issue
No description provided.
Delete Branch "eive/fsfw:mueller/clock-addition"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Can you replace calls to:
with this function and remove the CCSDSTime one? Afterwards it would be very easy to add an unittest. (If you don't have time I can do that).
I'm not sure about gmtime being available in any case. Some recommend gmtime_s but this seems to be broken at least for Windows (inverse parameter order) or unavailable in some cases.
See https://en.cppreference.com/w/c/chrono/gmtime
Hmm I didn't even see that there already is an (empty) implementation. Would have been interesting to see why this is tricky according to the comment. I implemented the suggestions and added an informative comment about the Windows CRT incompatibility. Theoretically shouldn't be an issue since
gmtime_s
is not used.It would be great if you could add unittests :)
added an additional conversion functionto WIP: dded an additional conversion functionWIP: dded an additional conversion functionto WIP: added an additional conversion functionI wrote some of the code myself
WIP: added an additional conversion functionto Added an additional conversion function(My review is subjective as I have written parts of this code.)