[RFA 3/3] Linespec rewrite: New tests

Tom Tromey tromey@redhat.com
Mon Mar 26 18:27:00 GMT 2012


>>>>> "Keith" == Keith Seitz <keiths@redhat.com> writes:

Keith> This final patch, also dealing with testing, introduces several new
Keith> tests. While one or two of these add tests for new features, several
Keith> just add missing tests, like "break ::foo" (surprisingly /not/ tested
Keith> today).

This looks pretty good.

If these tests all pass against CVS gdb, I think this could go in
independently of the rewrite patch.

Keith> +set base "ls-dollar"
Keith> +set srcfile "$base.cc"
Keith> +set testfile "$base.exp"
Keith> +set exefile $base
Keith> +
Keith> +if {[skip_cplus_tests]} {
Keith> +    unsupported linespec.exp

Wrong text in the argument here.
I think the new trend, proposed in the not-too-distant past by Pedro
(IIRC), is to put some useful text in there instead of just the .exp
name, since that is redundant anyhow.

Keith> +set base ls-errs
Keith> +set srcfile "$base.c"
Keith> +set testfile "$base.exp"
Keith> +set exefile $base
Keith> +
Keith> +if {[skip_cplus_tests]} {
Keith> +    unsupported linespec.exp

Wrong text -- but there doesn't seem to be a need to examine
skip_cplus_tests here at all.

Tom



More information about the Gdb-patches mailing list