[Lldb-commits] [PATCH] D13158: Allow to construct CMIUtilString using std::string directly (MI)

Ilia K via lldb-commits lldb-commits at lists.llvm.org
Thu Sep 24 23:19:47 PDT 2015


ki.stfu created this revision.
ki.stfu added reviewers: abidh, brucem.
ki.stfu added subscribers: abidh, brucem, lldb-commits.

Allow to construct CMIUtilString using std::string directly (MI)

This patch cleans up lldb-mi code, and serves to simplify
the following case:
```
  std::string strGoodbye = "!Hello";
  CMIUtilString strHello = strGoodbye.substr(1).c_str();
```

With CMIUtilString(std::string) we can omit .c_str():
```
  std::string strGoodbye = "!Hello";
  CMIUtilString strHello = strGoodbye.substr(1);
```

http://reviews.llvm.org/D13158

Files:
  tools/lldb-mi/MICmdArgContext.cpp
  tools/lldb-mi/MICmdArgValOptionLong.cpp
  tools/lldb-mi/MICmdArgValOptionShort.cpp
  tools/lldb-mi/MICmdArgValThreadGrp.cpp
  tools/lldb-mi/MICmdCmdData.cpp
  tools/lldb-mi/MICmdCmdVar.cpp
  tools/lldb-mi/MICmdInterpreter.cpp
  tools/lldb-mi/MICmnLLDBDebugger.cpp
  tools/lldb-mi/MICmnLLDBDebuggerHandleEvents.cpp
  tools/lldb-mi/MICmnLLDBUtilSBValue.cpp
  tools/lldb-mi/MIDriver.cpp
  tools/lldb-mi/MIUtilString.cpp
  tools/lldb-mi/MIUtilString.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D13158.35702.patch
Type: text/x-patch
Size: 11532 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150925/6b105d82/attachment.bin>


More information about the lldb-commits mailing list