[Lldb-commits] [PATCH] D20436: Clean up vestigial remnants of locking primitives

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Thu May 19 10:13:32 PDT 2016

clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.

We should think about converting everyone over to using std::chrono::microseconds for any timeout parameters that are currently "uint32_t timeout_usec". But we can do that in a future change.

Comment at: source/Core/Communication.cpp:163-165
@@ -169,3 +162,5 @@
         EventSP event_sp;
-        while (listener_sp->WaitForEvent (timeout_time.IsValid() ? &timeout_time : nullptr, event_sp))
+        while (listener_sp->WaitForEvent((timeout_usec == UINT32_MAX) ? std::chrono::microseconds(0)
+                                                                      : std::chrono::microseconds(timeout_usec),
+                                         event_sp))
Should we convert the "timeout_usec" argument over to be a std::chrono::microseconds type?



More information about the lldb-commits mailing list