This is the mail archive of the
gdb@sources.redhat.com
mailing list for the GDB project.
Re: [rfc] Completion tests failing
- To: drow at mvista dot com
- Subject: Re: [rfc] Completion tests failing
- From: "Eli Zaretskii" <eliz at is dot elta dot co dot il>
- Date: Thu, 27 Sep 2001 10:39:49 +0200
- CC: gdb at sources dot redhat dot com
- References: <20010926211336.A31592@nevyn.them.org>
- Reply-to: Eli Zaretskii <eliz at is dot elta dot co dot il>
> Date: Wed, 26 Sep 2001 21:13:36 -0400
> From: Daniel Jacobowitz <drow@mvista.com>
>
> The test failing for me right now is completion for 'p "a<tab>'. The
> comments, and the testsuite, say that this is a string literal and should
> not be completed.
This is no longer true, so the test suite should be changed.
> The code tries to complete it as a location, though, and
> decides to run it through location_completer. make_symbol_completion_list
> notices the stray ", and decides not to allow completion; but
> make_source_files_completion_list does no such thing.
This is intended behavior, I think: source files can have quotes in
them.
> Which behavior would people prefer? I can update the testsuite to match.
> I'd also like to add expressions like:
>
> -re "^p \"a\[^\"\]*\"$" { fail "complete 'p \"a'" }
>
> if it is not supposed to complete; that is, if gdb completes it to something
> and supplies the trailing quote, fail now, instead of waiting for timeout
> and failing the next dozen tests too.
Given the recent changes to completion, I really don't see any sense
in testing this. I'd suggest to delete the test.