[RFA] completer test [was Re: [RFC] Cleanup for make_source_files_completion_list]
Sergio Durigan Junior
sergiodj@redhat.com
Wed May 15 22:33:00 GMT 2013
Hi Keith,
I know you have already committed the patch, but I just would like to
ask a few things.
On Wednesday, May 15 2013, Keith Seitz wrote:
> diff --git a/gdb/testsuite/gdb.base/filesym.exp b/gdb/testsuite/gdb.base/filesym.exp
> new file mode 100644
> index 0000000..c9e9c20
> --- /dev/null
> +++ b/gdb/testsuite/gdb.base/filesym.exp
> +# Test completion list for "filesy". We expect the immediate result to
> +# complete to "filesym"; completing again, we expect the symbol name and the
> +# filename (in that order).
> +
> +send_gdb "break filesy\t"
> +gdb_expect {
> + -re "m\$" {
> + pass "complete on \"filesy\""
> +
> + # Now ask for the completion list
> + send_gdb "\t\t"
> + gdb_expect {
> + -re ".*filesym\[ \t\]+filesym.c.*$gdb_prompt break filesym\$" {
> + pass "completion list for \"filesym\""
> + }
> +
> + default {
> + fail "completion list for \"filesym\""
> + }
> + }
> + }
> +
> + default {
> + fail "complete on \"filesy\""
> + }
> +}
Wouldn't it be better to use the "complete" command? Here is what I see
when I use it:
(gdb) complete break filesy
break filesym
break filesym.c
Also, ISTR "send_gdb" is deprecated, and one should use
"gdb_test_multiple" instead. WDYT?
Thanks,
--
Sergio
More information about the Gdb-patches
mailing list