================
@@ -85,7 +85,8 @@ class JSONTransport {
 
   /// Reads the next message from the input stream.
   template <typename T>
-  llvm::Expected<T> Read(const std::chrono::microseconds &timeout) {
+  llvm::Expected<T>
+  Read(std::optional<std::chrono::microseconds> timeout = std::nullopt) {
----------------
ashgti wrote:

Since this type has moved into `lldb_private` now, should we use the `Timeout` 
helper? 
https://github.com/llvm/llvm-project/blob/main/lldb/include/lldb/Utility/Timeout.h#L28
 

https://github.com/llvm/llvm-project/pull/144610
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to