TODO: add a task for replacing chrono::duration with chrono::time_point

We should store event timestamps in C++ bindings in
::std::chrono::time_point objects rather than in
::std::chrono::duration. Add a task for that to TODO.

Suggested-by: Helmut Grohne <helmut.grohne@intenta.de>
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>

diff --git a/TODO b/TODO
index 320c470f04ded77ae3dffc34e69320ca133455ae..c7cae8eb9eadca96c3b7766305fa9a2b8f48bc23 100644 (file)
--- a/TODO
+++ b/TODO
@@ -101,3 +101,9 @@ of methods, properties should be used wherever possible (chip's name and label,
 line's direction, active-state etc.), we should probably store the line objects
 within the chip instead of creating them everytime get_line() is called and
 several more.
+
+* use ::std::chrono::time_point instead of ::std::chrono:duration in C++
+  bindings
+
+Duration is a time interval so it's not the best candidate for storing
+timestamps. Switch to using time_point where applicable.