summaryrefslogtreecommitdiffstats
path: root/lldb
diff options
context:
space:
mode:
authorRaphael Isemann <teemperor@gmail.com>2019-09-25 12:55:30 +0000
committerRaphael Isemann <teemperor@gmail.com>2019-09-25 12:55:30 +0000
commit7f9ac3372ccb85fbe652276d4081c9c64dfb3836 (patch)
treececbb5ac5a4bf33d0329dc0a31e40c26370b5048 /lldb
parentc00627f4dc9404756d643b0db3c5347455c78e4b (diff)
downloadbcm5719-llvm-7f9ac3372ccb85fbe652276d4081c9c64dfb3836.tar.gz
bcm5719-llvm-7f9ac3372ccb85fbe652276d4081c9c64dfb3836.zip
[lldb][NFC] Remove CompletionRequest::GetCursorArgument and GetRawLineUntilCursor
They both return the same result as another function (GetCursorArgumentPrefix and GetRawLine). They were only added because the old API allowed to look (in theory) behind the cursor position which is no longer possible. llvm-svn: 372861
Diffstat (limited to 'lldb')
-rw-r--r--lldb/include/lldb/Utility/CompletionRequest.h9
-rw-r--r--lldb/source/Expression/REPL.cpp4
-rw-r--r--lldb/source/Interpreter/Options.cpp2
3 files changed, 4 insertions, 11 deletions
diff --git a/lldb/include/lldb/Utility/CompletionRequest.h b/lldb/include/lldb/Utility/CompletionRequest.h
index 96f202b4b6b..570f626ac54 100644
--- a/lldb/include/lldb/Utility/CompletionRequest.h
+++ b/lldb/include/lldb/Utility/CompletionRequest.h
@@ -103,9 +103,6 @@ public:
CompletionResult &result);
llvm::StringRef GetRawLine() const { return m_command; }
- llvm::StringRef GetRawLineUntilCursor() const {
- return m_command.substr(0, m_cursor_index);
- }
unsigned GetRawCursorPos() const { return m_raw_cursor_pos; }
@@ -192,12 +189,8 @@ public:
descriptions.GetStringAtIndex(i));
}
- llvm::StringRef GetCursorArgument() const {
- return GetParsedLine().GetArgumentAtIndex(GetCursorIndex());
- }
-
llvm::StringRef GetCursorArgumentPrefix() const {
- return GetCursorArgument().substr(0, m_cursor_char_position);
+ return GetParsedLine().GetArgumentAtIndex(GetCursorIndex());
}
private:
diff --git a/lldb/source/Expression/REPL.cpp b/lldb/source/Expression/REPL.cpp
index 79352513458..f7bd86391f5 100644
--- a/lldb/source/Expression/REPL.cpp
+++ b/lldb/source/Expression/REPL.cpp
@@ -453,7 +453,7 @@ void REPL::IOHandlerComplete(IOHandler &io_handler,
}
// Strip spaces from the line and see if we had only spaces
- if (request.GetRawLineUntilCursor().trim().empty()) {
+ if (request.GetRawLine().trim().empty()) {
// Only spaces on this line, so just indent
request.AddCompletion(m_indent_str);
return;
@@ -479,7 +479,7 @@ void REPL::IOHandlerComplete(IOHandler &io_handler,
}
current_code.append("\n");
- current_code += request.GetRawLineUntilCursor();
+ current_code += request.GetRawLine();
StringList matches;
int result = CompleteCode(current_code, matches);
diff --git a/lldb/source/Interpreter/Options.cpp b/lldb/source/Interpreter/Options.cpp
index e5fa84f6f2a..465271d2f96 100644
--- a/lldb/source/Interpreter/Options.cpp
+++ b/lldb/source/Interpreter/Options.cpp
@@ -698,7 +698,7 @@ bool Options::HandleOptionCompletion(CompletionRequest &request,
request.AddCompletion("--" + long_option.str(), opt.usage_text);
return true;
} else
- request.AddCompletion(request.GetCursorArgument());
+ request.AddCompletion(request.GetCursorArgumentPrefix());
return true;
} else {
// FIXME - not handling wrong options yet:
OpenPOWER on IntegriCloud