This adds the data formatters for chrono duration typedefs.
You probably copied & pasted this — this is no longer needed since every test function is now running in its own instance.
This is not guaranteed behavior and is highly specific on the actual compiler. I would probably just remove this first sets of tests.
can you change this to a function call like
It's not guaranteed that all compilers actually generate code for this line that results in a breakpoint opportunity.
Thanks for doing this!
Question to all: Should the summary string include the unit? lldb doesn't always show the type, so it could help comprehension if the unit is included. For example 60s instead of 60.
there's some regex grouping that doesn't seem to be needed.
I was wondering about that too, but I wasn't sure whether that was wanted. If we do that what do you prefer for micro seconds us or µs. The latter uses Unicode. The C++ Standard allows both http://eel.is/c++draft/time.duration.io.
I indeed copy-pasted the vector code; it's the first time I look at these formatters.
@Mordante @Michael137 This seems to fail on older versions of compiler/libcxx https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake-matrix/7247/ You can probably fix this by just requiring a minimum clang version in the tests