Repository navigation
Add std::chrono::time_point constructor overloads and tests - #50
yalagadapavankumar wants to merge 5 commits into
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
|
Hi teor, i thought using just checklist is more convenient, as it makes easier to what completed. Do you think it's better to include whole PR template? |
This comment was marked as resolved.
This comment was marked as resolved.
This is now fixed, and if people accidentally introduce themselves somewhere else, someone from the community will move their post to the right channel. |
teor2345
left a comment
There was a problem hiding this comment.
Sorry, the instructions weren't clear, we want to see 3-5 different overloaded function names, wrapping all the overloads for each name.
I have updated the PR template and the checklist at the top of this PR with the new wording.
You can choose whatever overloaded functions you like, but I suggest:
| #include <chrono> | ||
|
|
||
| using TestClock = std::chrono::system_clock; | ||
| using TestDuration = std::chrono::milliseconds; |
There was a problem hiding this comment.
This specific type alias makes the code harder to read, because it hides that the type is milliseconds.
Instead, delete this alias, and use the milliseconds type directly.
Please also rename TestTimePoint so it is clear it is in milliseconds.
There was a problem hiding this comment.
yes, i updated according to milliseconds as you suggested.
Along with these i added another two ( |
|
I implemented the constructors, round(), floor(), ceil() and time_point_cast(). |
Add C++
std::chrono::time_pointconstructors in Rust usingoverload!macro.Limitationsdoc sectionErgonomicsdoc sectionIncompatibilitiesdoc sectionWorkaroundsdoc sectionInexpressible Overloadsdoc section