This revision was automatically updated to reflect the committed changes.
enlight marked an inline comment as done.
Closed by commit rL284231: [LLDB-MI] Minor cleanup of CMICmnLLDBUtilSBValue
class (authored by enlight).
Changed prior to commit:
enlight marked an inline comment as done.
enlight added inline comments.
Comment at: tools/lldb-mi/MICmnLLDBUtilSBValue.cpp:116
if (!bOk)
-return m_pUnkwn;
+return kUnresolvedValue;
ki.stfu wrote:
> After couple of minutes I see the lack of logic
enlight updated this revision to Diff 74628.
enlight marked an inline comment as done.
enlight added a comment.
Rename `kUnresolvedValue` back to `kUknownValue`.
Repository:
rL LLVM
https://reviews.llvm.org/D25592
Files:
tools/lldb-mi/MICmnLLDBUtilSBValue.cpp
ki.stfu requested changes to this revision.
ki.stfu added inline comments.
This revision now requires changes to proceed.
Comment at: tools/lldb-mi/MICmnLLDBUtilSBValue.cpp:116
if (!bOk)
-return m_pUnkwn;
+return kUnresolvedValue;
After couple of
ki.stfu accepted this revision.
ki.stfu added inline comments.
This revision is now accepted and ready to land.
Comment at: tools/lldb-mi/MICmnLLDBUtilSBValue.cpp:21-22
+static const char *kUnknownValue = "??";
+static const char *kCompositeValuePlaceholder = "{...}";
+
enlight added inline comments.
Comment at: tools/lldb-mi/MICmnLLDBUtilSBValue.cpp:21-22
+static const char *kUnknownValue = "??";
+static const char *kCompositeValuePlaceholder = "{...}";
+
ki.stfu wrote:
> use unnamed namespace here
I would like to use an
ki.stfu requested changes to this revision.
ki.stfu added inline comments.
This revision now requires changes to proceed.
Comment at: tools/lldb-mi/MICmnLLDBUtilSBValue.cpp:21-22
+static const char *kUnknownValue = "??";
+static const char *kCompositeValuePlaceholder =