dawn created this revision. dawn added reviewers: abidh, brucem, ki.stfu. dawn added a subscriber: lldb-commits. dawn set the repository for this revision to rL LLVM.
This fixes an off-by-one bug in CMICmdCmdDataInfoLine::Acknowledge. Given: LineEntry: \[0x0000000100000f37-0x0000000100000f45\): /path/to/file:123:1 -data-info-line would report the line as 12, omitting the last digit. Repository: rL LLVM http://reviews.llvm.org/D11209 Files: tools/lldb-mi/MICmdCmdData.cpp Index: tools/lldb-mi/MICmdCmdData.cpp =================================================================== --- tools/lldb-mi/MICmdCmdData.cpp +++ tools/lldb-mi/MICmdCmdData.cpp @@ -1789,7 +1789,7 @@ // ^ -- line const size_t nLineStartPos = nFileEndPos + 1; const size_t nLineEndPos = rLine.find(':', nLineStartPos); - const size_t nLineLen = nLineEndPos != std::string::npos ? nLineEndPos - nLineStartPos - 1 + const size_t nLineLen = nLineEndPos != std::string::npos ? nLineEndPos - nLineStartPos : std::string::npos; const CMIUtilString strLine(rLine.substr(nLineStartPos, nLineLen).c_str()); const CMICmnMIValueConst miValueConst4(strLine);
Index: tools/lldb-mi/MICmdCmdData.cpp =================================================================== --- tools/lldb-mi/MICmdCmdData.cpp +++ tools/lldb-mi/MICmdCmdData.cpp @@ -1789,7 +1789,7 @@ // ^ -- line const size_t nLineStartPos = nFileEndPos + 1; const size_t nLineEndPos = rLine.find(':', nLineStartPos); - const size_t nLineLen = nLineEndPos != std::string::npos ? nLineEndPos - nLineStartPos - 1 + const size_t nLineLen = nLineEndPos != std::string::npos ? nLineEndPos - nLineStartPos : std::string::npos; const CMIUtilString strLine(rLine.substr(nLineStartPos, nLineLen).c_str()); const CMICmnMIValueConst miValueConst4(strLine);
_______________________________________________ lldb-commits mailing list lldb-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits