teemperor added inline comments.
================ Comment at: lldb/include/lldb/Utility/Stream.h:417 + write(colorcode, len); + } + return *this; ---------------- teemperor wrote: > Nit: This can all be `PutCString(colorcode)` (which handles the nullptr check > and the strlen). Same as below. Actually nvm, I thought this is in the Stream class (not our own raw_ostream subclass). Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81058/new/ https://reviews.llvm.org/D81058 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits