This revision was not accepted when it landed; it landed in state "Needs
Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rL367308: [lldb] Fix crash when tab-completing in multi-line
expr (authored by teemperor, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D64995?vs=212163&id=212317#toc
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64995/new/
https://reviews.llvm.org/D64995
Files:
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/Makefile
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/TestMultilineCompletion.py
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/main.c
lldb/trunk/source/Core/IOHandler.cpp
Index: lldb/trunk/source/Core/IOHandler.cpp
===================================================================
--- lldb/trunk/source/Core/IOHandler.cpp
+++ lldb/trunk/source/Core/IOHandler.cpp
@@ -233,7 +233,7 @@
matches, descriptions);
case Completion::Expression: {
CompletionResult result;
- CompletionRequest request(current_line, current_line - cursor,
+ CompletionRequest request(current_line, cursor - current_line,
skip_first_n_matches, max_matches, result);
CommandCompletions::InvokeCommonCompletionCallbacks(
io_handler.GetDebugger().GetCommandInterpreter(),
Index:
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/main.c
===================================================================
---
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/main.c
+++
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/main.c
@@ -0,0 +1,5 @@
+int main(int argc, char **argv) {
+ lldb_enable_attach();
+ int to_complete = 0;
+ return to_complete;
+}
Index:
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/TestMultilineCompletion.py
===================================================================
---
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/TestMultilineCompletion.py
+++
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/TestMultilineCompletion.py
@@ -0,0 +1,52 @@
+"""
+Test completion for multiline expressions.
+"""
+
+import lldb
+from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+
+class MultilineCompletionTest(TestBase):
+
+ mydir = TestBase.compute_mydir(__file__)
+ NO_DEBUG_INFO_TESTCASE = True
+
+ def setUp(self):
+ TestBase.setUp(self)
+ self.source = 'main.c'
+
+ def expect_string(self, string):
+ import pexpect
+ """This expects for "string", with timeout & EOF being test fails."""
+ try:
+ self.child.expect_exact(string)
+ except pexpect.EOF:
+ self.fail("Got EOF waiting for '%s'" % (string))
+ except pexpect.TIMEOUT:
+ self.fail("Timed out waiting for '%s'" % (string))
+
+ @expectedFailureAll(
+ oslist=["windows"],
+ bugnumber="llvm.org/pr22274: need a pexpect replacement for windows")
+ def test_basic_completion(self):
+ """Test that we can complete a simple multiline expression"""
+ self.build()
+ self.setTearDownCleanup()
+
+ import pexpect
+ exe = self.getBuildArtifact("a.out")
+ prompt = "(lldb) "
+
+ run_commands = ' -o "b main" -o "r"'
+ self.child = pexpect.spawn(
+ '%s %s %s %s' %
+ (lldbtest_config.lldbExec, self.lldbOption, run_commands, exe))
+ child = self.child
+
+ self.expect_string(prompt)
+ self.child.sendline("expr")
+ self.expect_string("terminate with an empty line to evaluate")
+ self.child.send("to_\t")
+ self.expect_string("to_complete")
+
+ self.deletePexpectChild()
Index:
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/Makefile
===================================================================
---
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/Makefile
+++
lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/Makefile
@@ -0,0 +1,3 @@
+LEVEL = ../../make
+C_SOURCES := main.c
+include $(LEVEL)/Makefile.rules
Index: lldb/trunk/source/Core/IOHandler.cpp
===================================================================
--- lldb/trunk/source/Core/IOHandler.cpp
+++ lldb/trunk/source/Core/IOHandler.cpp
@@ -233,7 +233,7 @@
matches, descriptions);
case Completion::Expression: {
CompletionResult result;
- CompletionRequest request(current_line, current_line - cursor,
+ CompletionRequest request(current_line, cursor - current_line,
skip_first_n_matches, max_matches, result);
CommandCompletions::InvokeCommonCompletionCallbacks(
io_handler.GetDebugger().GetCommandInterpreter(),
Index: lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/main.c
===================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/main.c
+++ lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/main.c
@@ -0,0 +1,5 @@
+int main(int argc, char **argv) {
+ lldb_enable_attach();
+ int to_complete = 0;
+ return to_complete;
+}
Index: lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/TestMultilineCompletion.py
===================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/TestMultilineCompletion.py
+++ lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/TestMultilineCompletion.py
@@ -0,0 +1,52 @@
+"""
+Test completion for multiline expressions.
+"""
+
+import lldb
+from lldbsuite.test.decorators import *
+from lldbsuite.test.lldbtest import *
+
+class MultilineCompletionTest(TestBase):
+
+ mydir = TestBase.compute_mydir(__file__)
+ NO_DEBUG_INFO_TESTCASE = True
+
+ def setUp(self):
+ TestBase.setUp(self)
+ self.source = 'main.c'
+
+ def expect_string(self, string):
+ import pexpect
+ """This expects for "string", with timeout & EOF being test fails."""
+ try:
+ self.child.expect_exact(string)
+ except pexpect.EOF:
+ self.fail("Got EOF waiting for '%s'" % (string))
+ except pexpect.TIMEOUT:
+ self.fail("Timed out waiting for '%s'" % (string))
+
+ @expectedFailureAll(
+ oslist=["windows"],
+ bugnumber="llvm.org/pr22274: need a pexpect replacement for windows")
+ def test_basic_completion(self):
+ """Test that we can complete a simple multiline expression"""
+ self.build()
+ self.setTearDownCleanup()
+
+ import pexpect
+ exe = self.getBuildArtifact("a.out")
+ prompt = "(lldb) "
+
+ run_commands = ' -o "b main" -o "r"'
+ self.child = pexpect.spawn(
+ '%s %s %s %s' %
+ (lldbtest_config.lldbExec, self.lldbOption, run_commands, exe))
+ child = self.child
+
+ self.expect_string(prompt)
+ self.child.sendline("expr")
+ self.expect_string("terminate with an empty line to evaluate")
+ self.child.send("to_\t")
+ self.expect_string("to_complete")
+
+ self.deletePexpectChild()
Index: lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/Makefile
===================================================================
--- lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/Makefile
+++ lldb/trunk/packages/Python/lldbsuite/test/expression_command/multiline-completion/Makefile
@@ -0,0 +1,3 @@
+LEVEL = ../../make
+C_SOURCES := main.c
+include $(LEVEL)/Makefile.rules
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
