This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [commit] precord watchpoint tests
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Michael Snyder <msnyder at vmware dot com>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Mon, 23 Nov 2009 17:25:43 -0500
- Subject: Re: [commit] precord watchpoint tests
- References: <4B0AEBBB.6050904@vmware.com>
> +# FIXME 'set exec-dir' command should give some output so we can test.
> +gdb_test "set exec-direction forward" "" "set forward"
This is really a minor comment, so feel free to ignore. IMO, I don't
necessarily think that "set exec-dir" should produce some output
(that is not consistent with quite a few of our "set ..." commands),
particularly if the sole purpose of that output is to allow the
testsuite to verify that the command worked. If testing is the only
point of output, then perhaps you should consider using gdb_test_multiple.
I am not 100% sure, but I think that this routine will allow you to test
that the command produced exactly no output.
It would be nice if we could just address the issue right now and be
done with it, rather than keeping duplicating FIXMEs like this. Either
way - whether we decide to add some output or not, taking care of the
FIXME shouldn't be too time consuming... (assuming gdb_test_multiple
works as I think it does).
--
Joel