This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [rfc] Revamp logical.exp
- From: mec dot gnu at mindspring dot com (Michael Elizabeth Chastain)
- To: cagney at gnu dot org, gdb-patches at sources dot redhat dot com
- Date: Wed, 14 Jul 2004 14:20:14 -0400 (EDT)
- Subject: Re: [rfc] Revamp logical.exp
The basic idea and the implementation are good.
I hand-checked all the data in the tables for correctness.
It all looks fine to me.
In the test name, could you include something grep'able.
So that if someone gets:
FAIL: logical.exp: evaluation of x == y, {x = 1, y = 0}
They can find the right gdb_test by grepping on the wordage
("evalation of").
Yes, I see that there are only two gdb_test's in the whole file,
but I like the test names to have searchable parts. This has
been essential to me when finding things in structs.exp.
Also please consider adding $val into the test name:
FAIL: logical.exp: evaluation of x == y; {x = 1, y = 0}; expected 0
I think that makes it easier to see whether the test is written
correctly or not, especially when someone goes to add 50 new tests.
And, as always ... did you run it somewhere?
Michael C
2004-07-14 Andrew Cagney <cagney@gnu.org>
* gdb.base/logical.exp: Update copyright.
(evaluate): New procedure. Use to re-implement tests using
several tables.