[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