Details
-
Epic
-
Resolution: Unresolved
-
Not Evaluated
-
None
-
None
-
None
-
'std::chrono first' approach
-
6754795e5 (dev), 2a495c259 (dev), 6f9ace568 (dev), e426a4e3f (dev), 0d29a406f (dev), 5cea5fc80 (dev), a0e3da262 (dev), c9f4c0d45 (dev), be3dd0115 (dev), 72d660843 (dev), 4f618bde0 (dev)
Description
As discussed here (The discussion on this topic is still open, and this task may be rejected, so far, this task is only a placeholder to track the progress of the work )
Change the API of all methods representing time from int/qint64 to std::chrono::milliseconds, this allows:
This makes the range issues a SEP. std::chrono::millisecond isn't required to support a full 64-bit range in count(), so we'll likely have a problem with supporting the full 64-bit range at the OS level ourselves, esp. when we have to convert back, e.g. because we're calling std::condition_variable API. By using chrono-first, we move any overflows into the user's code and our functions can be noexcept. ~ mmutz
Follow the porting guide from qsizetype epic.
Attachments
Issue Links
- resulted from
-
QTBUG-67383 Change interval type in QTimer to qint64/QDeadlineTimer
- Open
-
QTBUG-113544 Add QChronoTimer
- Closed