================ @@ -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