util: Use std::chrono for time getters #18282

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2003-timeChrono changing 1 files +11 −9

MarcoFalke commented at 4:14 PM on March 6, 2020: member

No description provided.
util: Use std::chrono for time getters fa03f63975
MarcoFalke commented at 4:15 PM on March 6, 2020: member

We may have to wait for C++20 to be able to use this and be absolutely sure this returns UTC

So closing for now unless there is reason to push this.
DrahtBot added the label Utils/log/libs on Mar 6, 2020
sanjaykdragon commented at 4:57 PM on March 6, 2020: contributor

We may have to wait for C++20 to be able to use this and be absolutely sure this returns UTC

So closing for now unless there is reason to push this.

Why do we need to wait until C++20 to use this? From what I have seen, the commit you made uses features from C++11 to C++17 (although we don't support C++17 yet, afaik)
MarcoFalke commented at 5:00 PM on March 6, 2020: member

https://en.cppreference.com/w/cpp/chrono/system_clock
MarcoFalke closed this on Mar 6, 2020
MarcoFalke deleted the branch on Mar 6, 2020
MarcoFalke commented at 8:46 PM on March 6, 2020: member

Or use the sanity check thingy from #9566
MarcoFalke added this to the "Later" column in a project

Contributors

Labels