[Lldb-commits] [PATCH] D127937: [lldb] Add RotatingLogHandler
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Jun 16 14:42:28 PDT 2022
JDevlieghere marked an inline comment as done.
JDevlieghere added inline comments.
================
Comment at: lldb/unittests/Utility/LogTest.cpp:112
+ handler.Dump(stream);
+ return stream.str();
+}
----------------
kastiglione wrote:
> minor: `return buffer;`
I did that on purpose to avoid the call to `flush` but looking at the `raw_string_ostream` the stream is unbuffered and `str` doesn't even call flush under the hood. TIL.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D127937/new/
https://reviews.llvm.org/D127937
More information about the lldb-commits
mailing list