From: David Carlton <carlton@kealia.com>
To: Andrew Cagney <ac131313@redhat.com>
Cc: Michael Elizabeth Chastain <mec@shout.net>,
jimb@redhat.com, gdb-patches@sources.redhat.com
Subject: Re: RFA: Don't include value of expression in pc-fp.exp test name
Date: Wed, 01 Oct 2003 15:49:00 -0000 [thread overview]
Message-ID: <yf2r81xklfo.fsf@hawaii.kealia.com> (raw)
In-Reply-To: <3F7AF34A.4070605@redhat.com> (Andrew Cagney's message of "Wed, 01 Oct 2003 11:31:22 -0400")
On Wed, 01 Oct 2003 11:31:22 -0400, Andrew Cagney <ac131313@redhat.com> said:
> As Michael well knows, supplemental information, such as which
> specific branch of a test passed or failed can be included in paren
> in the test message. Any analysis tools comparing test results
> needs to accomodate this convention.
Then check in some analysis tools that do this. Until you do that,
I'm going to stick with 'diff -u': it works fine for everything but
the threads test (where reordering is harder to get away with) and
pc-fp.exp. From my point of view, all that you've accomplished by
putting that hex string in pc-fp.exp is made it more likely that I'll
ignore the test, just like I do with print-threads.exp and
schedlock.exp.
I really don't understand the motivation behind putting random stuff
in parentheses and then complaining that people aren't ignoring it.
If you want people to ignore it, and even encourage people to use
tools which shield them from it, then why have it there in the first
place? I can see that making FAIL messages more verbose, especially
if the verbosity is in human-readable form. But PASS messages, with
hex strings? What am I supposed to do with that hex string?
Especially since I can get that data out of gdb.log if I really need
it.
David Carlton
carlton@kealia.com
next prev parent reply other threads:[~2003-10-01 15:49 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-09-30 17:49 Michael Elizabeth Chastain
2003-09-30 22:18 ` Jim Blandy
2003-10-01 15:31 ` Andrew Cagney
2003-10-01 15:49 ` David Carlton [this message]
-- strict thread matches above, loose matches on Subject: below --
2003-10-02 22:43 Michael Elizabeth Chastain
2003-10-01 19:06 Michael Elizabeth Chastain
2003-10-02 21:55 ` Andrew Cagney
2003-10-01 16:44 Michael Elizabeth Chastain
2003-10-01 18:29 ` Andrew Cagney
2003-10-01 19:33 ` David Carlton
2003-10-01 4:12 Michael Elizabeth Chastain
[not found] <200309291917.h8TJHLhn010447@duracef.shout.net>
2003-09-30 6:44 ` Jim Blandy
2003-09-30 15:36 ` David Carlton
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=yf2r81xklfo.fsf@hawaii.kealia.com \
--to=carlton@kealia.com \
--cc=ac131313@redhat.com \
--cc=gdb-patches@sources.redhat.com \
--cc=jimb@redhat.com \
--cc=mec@shout.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox