* Re: [patch/rfc] Cleanup store.exp
@ 2004-07-19 20:37 Michael Elizabeth Chastain
2004-07-26 20:56 ` Andrew Cagney
0 siblings, 1 reply; 6+ messages in thread
From: Michael Elizabeth Chastain @ 2004-07-19 20:37 UTC (permalink / raw)
To: cagney, gdb-patches
Did you test it?
Andrew, we go through this too much. Could you make some sort of note
which says: "Michael wants to see me say that I tested the darn thing,
every single time."
This patch is not approved. Please re-submit with a note that says
how you tested it.
Michael C
2004-07-19 Andrew Cagney <cagney@gnu.org>
* gdb.base/store.exp: Update copyright. Tweak indentation.
(check_set, up_set, check_struct, up_struct): Add a prefix to test
names, do not import gdb_prompt.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [patch/rfc] Cleanup store.exp
2004-07-19 20:37 [patch/rfc] Cleanup store.exp Michael Elizabeth Chastain
@ 2004-07-26 20:56 ` Andrew Cagney
2004-07-27 0:44 ` Michael Chastain
0 siblings, 1 reply; 6+ messages in thread
From: Andrew Cagney @ 2004-07-26 20:56 UTC (permalink / raw)
To: Michael Elizabeth Chastain; +Cc: gdb-patches
> Did you test it?
Obviously.
What isn't so obvious is the system used, but even there you'd know the
answer - GNU/Linux amd64 and ppc NetBSD :-)
> Andrew, we go through this too much. Could you make some sort of note
> which says: "Michael wants to see me say that I tested the darn thing,
> every single time."
Me individually or everyone generally? I see:
20 Michael Chastain
15 Andrew Cagney
5 Corinna Vinschen
4 Jim Blandy
2 Jeff Johnston
1 Bob Rossi
1 Mark Kettenis
1 Martin Hunt
so it doesn't suprise me that my name has come up more often than
everyone else. The requirement here is new and foreign so it is no
suprize that you're having trouble getting people to comply.
Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [patch/rfc] Cleanup store.exp
2004-07-26 20:56 ` Andrew Cagney
@ 2004-07-27 0:44 ` Michael Chastain
2004-07-27 15:45 ` Andrew Cagney
0 siblings, 1 reply; 6+ messages in thread
From: Michael Chastain @ 2004-07-27 0:44 UTC (permalink / raw)
To: cagney; +Cc: gdb-patches
mec> Did you test it?
ac> Obviously.
Then just say so.
As you know, I ask this of everybody. It's not just you. And it
shouldn't be too much of a surprise, because I said I would do this
during the comment period when I was nominated for this position.
Remember?
I hate to put it like this, but I can't find a better way:
you're the only person I'm having trouble getting to comply.
I did not even read this patch, so could you please re-submit,
and include the obligatory "yes I tested it", optional "sheesh!".
Michael C
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [patch/rfc] Cleanup store.exp
2004-07-27 0:44 ` Michael Chastain
@ 2004-07-27 15:45 ` Andrew Cagney
2004-07-27 16:11 ` Michael Chastain
0 siblings, 1 reply; 6+ messages in thread
From: Andrew Cagney @ 2004-07-27 15:45 UTC (permalink / raw)
To: Michael Chastain; +Cc: gdb-patches
> I hate to put it like this, but I can't find a better way:
> you're the only person I'm having trouble getting to comply.
Michael, look at the stats - you're reminding everyone. It's just that
I'm submitting far more patches than anyone else and, not unsuprisingly,
am being reminded more often. This is after all a very new and very
foreign requirement and one that will take time to get accustomed to.
Please show just a little tollerance of your peers.
shesh,
Andrew
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [patch/rfc] Cleanup store.exp
2004-07-27 15:45 ` Andrew Cagney
@ 2004-07-27 16:11 ` Michael Chastain
0 siblings, 0 replies; 6+ messages in thread
From: Michael Chastain @ 2004-07-27 16:11 UTC (permalink / raw)
To: cagney; +Cc: gdb-patches
> Michael, look at the stats - you're reminding everyone.
Yes. Generally when I tell people that I want to know how they tested
their patches, they just come back and tell me how they tested the
patch. And I don't have to keep telling the same person, over and over.
With you, it's like tug-of-war, over and over.
Michael C
^ permalink raw reply [flat|nested] 6+ messages in thread
* [patch/rfc] Cleanup store.exp
@ 2004-07-19 15:17 Andrew Cagney
0 siblings, 0 replies; 6+ messages in thread
From: Andrew Cagney @ 2004-07-19 15:17 UTC (permalink / raw)
To: gdb-patches
[-- Attachment #1: Type: text/plain, Size: 52 bytes --]
It's got duplicate test messages.
thoughts?
Andrew
[-- Attachment #2: diffs --]
[-- Type: text/plain, Size: 5795 bytes --]
2004-07-19 Andrew Cagney <cagney@gnu.org>
* gdb.base/store.exp: Update copyright. Tweak indentation.
(check_set, up_set, check_struct, up_struct): Add a prefix to test
names, do not import gdb_prompt.
Index: gdb.base/store.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.base/store.exp,v
retrieving revision 1.4
diff -p -u -r1.4 store.exp
--- gdb.base/store.exp 3 Jul 2003 13:55:22 -0000 1.4
+++ gdb.base/store.exp 19 Jul 2004 15:14:50 -0000
@@ -1,4 +1,6 @@
-# Copyright 2002, 2003 Free Software Foundation, Inc.
+# This testcase is part of GDB, the GNU debugger.
+
+# Copyright 2002, 2003, 2004 Free Software Foundation, Inc.
# This program is free software; you can redistribute it and/or modify
# it under the terms of the GNU General Public License as published by
@@ -14,9 +16,6 @@
# along with this program; if not, write to the Free Software
# Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
-# Please email any bugs, comments, and/or additions to this file to:
-# bug-gdb@gnu.org
-
if $tracelevel {
strace $tracelevel
}
@@ -31,7 +30,7 @@ set testfile "store"
set srcfile ${testfile}.c
set binfile ${objdir}/${subdir}/${testfile}
if { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable {debug}] != "" } {
- gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
+ gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
}
if [get_compiler_info ${binfile}] {
@@ -55,16 +54,24 @@ if ![runto_main] then {
#
proc check_set { t l r new add } {
- global gdb_prompt
+ set prefix "var ${t} l"
gdb_test "tbreak wack_${t}"
- gdb_test "continue" "register ${t} l = u, r = v;" "continue to wack_${t}"
- gdb_test "next" "l = add_${t} .l, r.;" "next ${t}"
- gdb_test "print l" " = ${l}" "print old l - ${t}"
- gdb_test "print r" " = ${r}" "print old r - ${t}"
- gdb_test "set variable l = 4"
- gdb_test "print l" " = ${new}" "print new l - ${t}"
- gdb_test "next" "return l \\+ r;"
- gdb_test "print l" " = ${add}" "print add - ${t}"
+ gdb_test "continue" "register ${t} l = u, r = v;" \
+ "continue to wack_${t}"
+ gdb_test "next" "l = add_${t} .l, r.;" \
+ "${prefix}; next ${t}"
+ gdb_test "print l" " = ${l}" \
+ "${prefix}; print old l, expecting ${l}"
+ gdb_test "print r" " = ${r}" \
+ "${prefix}; print old r, expecting ${r}"
+ gdb_test "set variable l = 4" "" \
+ "${prefix}; setting l to 4"
+ gdb_test "print l" " = ${new}" \
+ "${prefix}; print new l, expecting ${l}"
+ gdb_test "next" "return l \\+ r;" \
+ "${prefix}; next over add call"
+ gdb_test "print l" " = ${add}" \
+ "${prefix}; print incremented l, expecting ${add}"
}
check_set "charest" "-1 .*" "-2 .*" "4 ..004." "2 ..002."
@@ -79,14 +86,20 @@ check_set "doublest" "-1" "-2" "4" "2"
#
proc up_set { t l r new } {
- global gdb_prompt
+ set prefix "upvar ${t} l"
gdb_test "tbreak add_${t}"
- gdb_test "continue" "return u . v;" "continue to add_${t}"
- gdb_test "up" "l = add_${t} .l, r.;" "up ${t}"
- gdb_test "print l" " = ${l}" "up print old l - ${t}"
- gdb_test "print r" " = ${r}" "up print old r - ${t}"
- gdb_test "set variable l = 4"
- gdb_test "print l" " = ${new}" "up print new l - ${t}"
+ gdb_test "continue" "return u . v;" \
+ "continue to add_${t}"
+ gdb_test "up" "l = add_${t} .l, r.;" \
+ "${prefix}; up"
+ gdb_test "print l" " = ${l}" \
+ "${prefix}; print old l, expecting ${l}"
+ gdb_test "print r" " = ${r}" \
+ "${prefix}; print old r, expecting ${r}"
+ gdb_test "set variable l = 4" "" \
+ "${prefix}; set l to 4"
+ gdb_test "print l" " = ${new}" \
+ "${prefix}; print new l, expecting ${new}"
}
up_set "charest" "-1 .*" "-2 .*" "4 ..004."
@@ -101,14 +114,18 @@ up_set "doublest" "-1" "-2" "4"
#
proc check_struct { t old new } {
- global gdb_prompt
+ set prefix "var struct ${t} u"
gdb_test "tbreak wack_struct_${t}"
gdb_test "continue" "int i; register struct s_${t} u = z_${t};" \
- "continue set struct ${t}"
- gdb_test "next 2" "add_struct_${t} .u.;"
- gdb_test "print u" " = ${old}" "old check struct ${t}"
- gdb_test "set variable u = s_${t}"
- gdb_test "print u" " = ${new}" "new check struct ${t}"
+ "continue to wack_struct_${t}"
+ gdb_test "next 2" "add_struct_${t} .u.;" \
+ "${prefix}; next to add_struct_${t} call"
+ gdb_test "print u" " = ${old}" \
+ "${prefix}; print old u, expecting ${old}"
+ gdb_test "set variable u = s_${t}" "" \
+ "${prefix}; set u to s_${t}"
+ gdb_test "print u" " = ${new}" \
+ "${prefix}; print new u, expecting ${new}"
}
check_struct "1" "{s = \\{0}}" "{s = \\{1}}"
@@ -117,14 +134,18 @@ check_struct "3" "{s = \\{0, 0, 0}}" "{s
check_struct "4" "{s = \\{0, 0, 0, 0}}" "{s = \\{1, 2, 3, 4}}"
proc up_struct { t old new } {
- global gdb_prompt
+ set prefix "up struct ${t} u"
gdb_test "tbreak add_struct_${t}"
gdb_test "continue" "for .i = 0; i < sizeof .s. / sizeof .s.s.0..; i..." \
- "continue up struct ${t}"
- gdb_test "up" "u = add_struct_${t} .u.;" "up struct ${t}"
- gdb_test "print u" " = ${old}" "old up struct ${t}"
- gdb_test "set variable u = s_${t}"
- gdb_test "print u" " = ${new}" "new up struct ${t}"
+ "continue to add_struct_${t}"
+ gdb_test "up" "u = add_struct_${t} .u.;" \
+ "${prefix}; up"
+ gdb_test "print u" " = ${old}" \
+ "${prefix}; print old u, expecting ${old}"
+ gdb_test "set variable u = s_${t}" "" \
+ "${prefix}; set u to s_${t}"
+ gdb_test "print u" " = ${new}" \
+ "${prefix}; print new u, expecting ${new}"
}
up_struct "1" "{s = \\{0}}" "{s = \\{1}}"
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2004-07-27 16:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-07-19 20:37 [patch/rfc] Cleanup store.exp Michael Elizabeth Chastain
2004-07-26 20:56 ` Andrew Cagney
2004-07-27 0:44 ` Michael Chastain
2004-07-27 15:45 ` Andrew Cagney
2004-07-27 16:11 ` Michael Chastain
-- strict thread matches above, loose matches on Subject: below --
2004-07-19 15:17 Andrew Cagney
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox