Re: [PATCH] Add autocompletion for convenience vars in print and set

[Andrew Burgess <aburgess at broadcom dot com>]

On 22/05/2014 4:17 PM, Daniel Gutson wrote:
> Second version.
> Comments below:
> 
> On Tue, May 20, 2014 at 1:36 PM, Tom Tromey <tromey@redhat.com> wrote:
>>>>>>> "Daniel" == Daniel Gutson <daniel.gutson@tallertechnologies.com> writes:
>>
>> Daniel> I could not find any testsuite where to add tests for this; if
>> Daniel> there are, please let me know.
>>
>> See testsuite/gdb.base/completion.exp
> 
> Thanks, I ran all the tests and passed. I did not add a new test
> case though since I didn't check how to add a new convenience var
> and undefine it later from the test framework. (Should I try harder?)

I'm not a maintainer, but you'll probably be asked for some tests of 
this (really useful) feature.

I don't think you need to worry about removing any convenience 
variables you create, each .exp test file starts gdb afresh, 
and sometime (look for clean_restart) gdb is restarted within 
a single .exp file.

For this feature you can probably get away with just adding some 
new tests to the end of the completion.exp file, I've included an
example test in this mail, but you'll probably want some more.

thanks,
Andrew


diff --git a/gdb/testsuite/gdb.base/completion.exp b/gdb/testsuite/gdb.base/completion.exp
index 2608309..c028548 100644
--- a/gdb/testsuite/gdb.base/completion.exp
+++ b/gdb/testsuite/gdb.base/completion.exp
@@ -747,6 +747,27 @@ gdb_test_multiple "" "$test" {
     }
 }
 
+set test "complete '\$cv_'"
+gdb_test_no_output "set \$cv_aaa = 4" \
+    "Create convenience variable \$cv_aaa"
+send_gdb "p \$cv_"
+gdb_test_multiple "" "$test" {
+    -re "^p \\\$cv_" {
+	send_gdb "\t"
+	gdb_test_multiple "" "$test" {
+	    -re "aaa $" {
+		send_gdb "\n"
+		gdb_test_multiple "" "$test" {
+		    -re "\\\$\[0-9\]+ = 4.*$gdb_prompt $" {
+			pass "$test"
+		    }
+		}
+	    }
+	}
+    }
+}
+
+
 # Restore globals modified in this test...
 set timeout $oldtimeout1