clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.
Please add Xcode changes I sent you offline and this will be good to go.
https://reviews.llvm.org/D29964
___
lldb-commits mailing
zturner updated this revision to Diff 88626.
https://reviews.llvm.org/D29964
Files:
lldb/include/lldb/Host/PseudoTerminal.h
lldb/include/lldb/Utility/PseudoTerminal.h
lldb/include/lldb/Utility/VASPrintf.h
lldb/source/Host/CMakeLists.txt
lldb/source/Host/common/PseudoTerminal.cpp
labath added inline comments.
Comment at: lldb/source/Utility/VASprintf.cpp:18
+ va_list args) {
+ llvm::SmallString<16> error("");
+
It doesn't look like you should need to allocate a stack object with the string
every time. Can't
zturner created this revision.
Herald added subscribers: mgorny, srhines, danalbert.
This patch finally gets `lldbUtility` to be dependency-free. This was done
through the following changes:
1. `PseudoTerminal.cpp` : `Utility` -> `Host`
2. Introduce a `vasprintf` / `vsnprintf` wrapper into