From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 27134 invoked by alias); 6 Nov 2003 06:44:09 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 27125 invoked from network); 6 Nov 2003 06:44:09 -0000 Received: from unknown (HELO concert.shout.net) (204.253.184.25) by sources.redhat.com with SMTP; 6 Nov 2003 06:44:09 -0000 Received: from duracef.shout.net (duracef.shout.net [204.253.184.12]) by concert.shout.net (8.12.10/8.12.10) with ESMTP id hA66i797023037; Thu, 6 Nov 2003 00:44:07 -0600 Received: from duracef.shout.net (localhost [127.0.0.1]) by duracef.shout.net (8.12.10/8.12.9) with ESMTP id hA66i7fa029406; Thu, 6 Nov 2003 00:44:07 -0600 Received: (from mec@localhost) by duracef.shout.net (8.12.10/8.12.9/Submit) id hA66i7hx029405; Thu, 6 Nov 2003 01:44:07 -0500 Date: Thu, 06 Nov 2003 06:44:00 -0000 From: Michael Elizabeth Chastain Message-Id: <200311060644.hA66i7hx029405@duracef.shout.net> To: gdb-patches@sources.redhat.com, jimb@redhat.com Subject: Re: RFA: give store.exp tests unique names X-SW-Source: 2003-11/txt/msg00077.txt.bz2 > Doesn't Michael C.'s system detect duplicate test names? > Perhaps I am doing something wrong. When my Crunch.pl sees a duplicate, it just renames the duplicate to "gdb.foo/bar {2}", and then "gdb.foo/bar {3}", and so on. Crunch.pl prints a count of duplicates on stdout but I don't even pay attention to that any more. Most of the duplicates are in tests that always PASS so it's low priority to get them cleaned up. Did this one in store.exp cause an actual problem? If it did, I'm all for stomping on it. No comments on your specific patch, I'm too tired. Michael C 2003-11-05 Jim Blandy * gdb.base/store.exp: Make sure every test has a unique name. Index: gdb/testsuite/gdb.base/store.exp =================================================================== RCS file: /cvs/cvsfiles/devo/gdb/testsuite/gdb.base/store.exp,v retrieving revision 1.4 diff -c -r1.4 store.exp *** gdb/testsuite/gdb.base/store.exp 14 Aug 2003 22:44:06 -0000 1.4 --- gdb/testsuite/gdb.base/store.exp 6 Nov 2003 04:47:03 -0000 *************** *** 61,69 **** 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}" } --- 61,69 ---- 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" "" "set l - ${t}" gdb_test "print l" " = ${new}" "print new l - ${t}" ! gdb_test "next" "return l \\+ r;" "next over call - ${t}" gdb_test "print l" " = ${add}" "print add - ${t}" } *************** *** 85,91 **** 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}" } --- 85,91 ---- 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" "" "up set l - ${t}" gdb_test "print l" " = ${new}" "up print new l - ${t}" } *************** *** 105,113 **** 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}" } --- 105,113 ---- 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.;" "check struct next ${t}" gdb_test "print u" " = ${old}" "old check struct ${t}" ! gdb_test "set variable u = s_${t}" "" "set check struct ${t}" gdb_test "print u" " = ${new}" "new check struct ${t}" } *************** *** 123,129 **** "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}" } --- 123,129 ---- "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}" "" "set up struct ${t}" gdb_test "print u" " = ${new}" "new up struct ${t}" }