From: Yao Qi <qiyaoltc@gmail.com>
To: Luis Machado <lgustavo@codesourcery.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH v2 2/8] Fix test names starting with uppercase using gdb_test on a single line.
Date: Sun, 27 Nov 2016 16:48:00 -0000 [thread overview]
Message-ID: <20161127164749.4sdm7r47f7nbe6se@localhost> (raw)
In-Reply-To: <1480107244-1484-3-git-send-email-lgustavo@codesourcery.com>
On Fri, Nov 25, 2016 at 02:53:58PM -0600, Luis Machado wrote:
> This fixes offender testcases that have test names starting with uppercase
> when using gdb_test in a single line construct.
>
> gdb/testsuite/ChangeLog
> 2016-11-25 Luis Machado <lgustavo@codesourcery.com>
>
> Fix test names starting with uppercase throughout the files.
>
> * gdb/testsuite/gdb.arch/i386-mpx-simple_segv.exp
> * gdb/testsuite/gdb.arch/i386-mpx.exp
> * gdb/testsuite/gdb.arch/i386-permbkpt.exp
> * gdb/testsuite/gdb.arch/pa-nullify.exp
> * gdb/testsuite/gdb.arch/powerpc-d128-regs.exp
> * gdb/testsuite/gdb.arch/vsx-regs.exp
> * gdb/testsuite/gdb.base/bfp-test.exp
> * gdb/testsuite/gdb.base/break.exp
> * gdb/testsuite/gdb.base/breakpoint-shadow.exp
> * gdb/testsuite/gdb.base/callfuncs.exp
> * gdb/testsuite/gdb.base/charset.exp
> * gdb/testsuite/gdb.base/commands.exp
> * gdb/testsuite/gdb.base/completion.exp
> * gdb/testsuite/gdb.base/dfp-test.exp
> * gdb/testsuite/gdb.base/echo.exp
> * gdb/testsuite/gdb.base/ending-run.exp
> * gdb/testsuite/gdb.base/eval.exp
> * gdb/testsuite/gdb.base/expand-psymtabs.exp
> * gdb/testsuite/gdb.base/float128.exp
> * gdb/testsuite/gdb.base/floatn.exp
> * gdb/testsuite/gdb.base/foll-exec-mode.exp
> * gdb/testsuite/gdb.base/gdb1056.exp
> * gdb/testsuite/gdb.base/gdb11531.exp
> * gdb/testsuite/gdb.base/kill-after-signal.exp
> * gdb/testsuite/gdb.base/multi-forks.exp
> * gdb/testsuite/gdb.base/overlays.exp
> * gdb/testsuite/gdb.base/pending.exp
> * gdb/testsuite/gdb.base/sepdebug.exp
> * gdb/testsuite/gdb.base/testenv.exp
> * gdb/testsuite/gdb.base/valgrind-db-attach.exp
> * gdb/testsuite/gdb.base/watch_thread_num.exp
> * gdb/testsuite/gdb.base/watchpoint-cond-gone.exp
> * gdb/testsuite/gdb.base/watchpoint.exp
> * gdb/testsuite/gdb.base/watchpoints.exp
> * gdb/testsuite/gdb.cp/arg-reference.exp
> * gdb/testsuite/gdb.cp/baseenum.exp
> * gdb/testsuite/gdb.cp/operator.exp
> * gdb/testsuite/gdb.cp/shadow.exp
> * gdb/testsuite/gdb.dwarf2/dw2-op-out-param.exp
> * gdb/testsuite/gdb.dwarf2/dw2-reg-undefined.exp
> * gdb/testsuite/gdb.go/chan.exp
> * gdb/testsuite/gdb.go/hello.exp
> * gdb/testsuite/gdb.go/integers.exp
> * gdb/testsuite/gdb.go/methods.exp
> * gdb/testsuite/gdb.go/package.exp
> * gdb/testsuite/gdb.guile/scm-parameter.exp
> * gdb/testsuite/gdb.guile/scm-progspace.exp
> * gdb/testsuite/gdb.guile/scm-value.exp
> * gdb/testsuite/gdb.mi/mi-pending.exp
> * gdb/testsuite/gdb.mi/user-selected-context-sync.exp
> * gdb/testsuite/gdb.multi/multi-attach.exp
> * gdb/testsuite/gdb.multi/tids.exp
> * gdb/testsuite/gdb.opt/clobbered-registers-O2.exp
> * gdb/testsuite/gdb.pascal/floats.exp
> * gdb/testsuite/gdb.pascal/integers.exp
> * gdb/testsuite/gdb.python/py-block.exp
> * gdb/testsuite/gdb.python/py-events.exp
> * gdb/testsuite/gdb.python/py-finish-breakpoint2.exp
> * gdb/testsuite/gdb.python/py-parameter.exp
> * gdb/testsuite/gdb.python/py-symbol.exp
> * gdb/testsuite/gdb.python/py-symtab.exp
> * gdb/testsuite/gdb.python/py-type.exp
> * gdb/testsuite/gdb.python/py-value.exp
> * gdb/testsuite/gdb.python/py-xmethods.exp
> * gdb/testsuite/gdb.python/python.exp
> * gdb/testsuite/gdb.reverse/break-precsave.exp
> * gdb/testsuite/gdb.reverse/consecutive-precsave.exp
> * gdb/testsuite/gdb.reverse/finish-precsave.exp
> * gdb/testsuite/gdb.reverse/i386-precsave.exp
> * gdb/testsuite/gdb.reverse/machinestate-precsave.exp
> * gdb/testsuite/gdb.reverse/sigall-precsave.exp
> * gdb/testsuite/gdb.reverse/solib-precsave.exp
> * gdb/testsuite/gdb.reverse/step-precsave.exp
> * gdb/testsuite/gdb.reverse/until-precsave.exp
> * gdb/testsuite/gdb.reverse/watch-precsave.exp
> * gdb/testsuite/gdb.server/ext-attach.exp
> * gdb/testsuite/gdb.server/ext-restart.exp
> * gdb/testsuite/gdb.server/ext-run.exp
> * gdb/testsuite/gdb.server/ext-wrapper.exp
> * gdb/testsuite/gdb.stabs/gdb11479.exp
> * gdb/testsuite/gdb.stabs/weird.exp
> * gdb/testsuite/gdb.threads/attach-many-short-lived-threads.exp
> * gdb/testsuite/gdb.threads/kill.exp
> * gdb/testsuite/gdb.threads/watchpoint-fork.exp
Drop "gdb/testsuite/".
> # test break at line number in file
> @@ -670,7 +670,7 @@ proc test_next_with_recursion {} {
> global decimal
> global binfile
>
> - gdb_test "kill" "" "kill program" "Kill the program being debugged.*y or n. $" "y"
> + gdb_test "kill" "" "kill program" "kill the program being debugged.*y or n. $" "y"
"Kill the program" is to be matched to the output of GDB, so this change
is not needed. There are some instances of this problem in this patch.
> delete_breakpoints
>
> gdb_test "break factorial" "Breakpoint $decimal at .*" "break at factorial"
>
> diff --git a/gdb/testsuite/gdb.base/kill-after-signal.exp b/gdb/testsuite/gdb.base/kill-after-signal.exp
> index 4181bec..69b27e7 100644
> --- a/gdb/testsuite/gdb.base/kill-after-signal.exp
> +++ b/gdb/testsuite/gdb.base/kill-after-signal.exp
> @@ -37,4 +37,4 @@ if ![runto_main] {
>
> gdb_test "continue" "Program received signal SIGUSR1, .*"
> gdb_test "stepi" "\r\nhandler .*"
> -gdb_test "kill" "^y" "kill" "Kill the program being debugged\\? \\(y or n\\) $" "y"
> +gdb_test "kill" "^y" "kill" "kill the program being debugged\\? \\(y or n\\) $" "y"
Likewise.
> # test break at function in file
> @@ -480,7 +480,7 @@ proc test_next_with_recursion {} {
> global decimal
> global binfile
>
> - gdb_test "kill" "" "kill program" "Kill the program being debugged.*y or n. $" "y"
> + gdb_test "kill" "" "kill program" "kill the program being debugged.*y or n. $" "y"
> delete_breakpoints
>
Likewise.
> # Explicitly kill the program so it doesn't dump core when we quit->detach.
> -gdb_test "kill" "" "kill program" "Kill the program being debugged.*y or n. $" "y"
> +gdb_test "kill" "" "kill program" "kill the program being debugged.*y or n. $" "y"
Likewise.
> -gdb_test "kill" "" "kill inferior 2" "Kill the program being debugged.*" "y"
> +gdb_test "kill" "" "kill inferior 2" "kill the program being debugged.*" "y"
> gdb_test "inferior 1" ".*Switching to inferior 1.*"
> -gdb_test "kill" "" "kill inferior 1" "Kill the program being debugged.*" "y"
> +gdb_test "kill" "" "kill inferior 1" "kill the program being debugged.*" "y"
Likewise.
> kill_wait_spawned_process $test_spawn_id1
> kill_wait_spawned_process $test_spawn_id2
> diff --git a/gdb/testsuite/gdb.multi/tids.exp b/gdb/testsuite/gdb.multi/tids.exp
> index 12ce98a..4808b0d 100644
> --- a/gdb/testsuite/gdb.multi/tids.exp
> +++ b/gdb/testsuite/gdb.multi/tids.exp
> @@ -409,7 +409,7 @@ if { ![skip_python_tests] } {
> # Remove the second inferior and confirm that GDB goes back to showing
> # single-number thread IDs.
> with_test_prefix "back to one inferior" {
> - gdb_test "kill inferior 2" "" "kill inferior 2" "Kill the program being debugged.*" "y"
> + gdb_test "kill inferior 2" "" "kill inferior 2" "kill the program being debugged.*" "y"
> gdb_test "thread 1.1" "Switching to thread 1\.1 .*"
> gdb_test "remove-inferior 2" ".*" "remove inferior 2"
>
Likewise.
> -gdb_test "continue" ".*exception did not finish.*" "FinishBreakpoint with exception thrown not caught"
> +gdb_test "continue" ".*exception did not finish.*" "finishBreakpoint with exception thrown not caught"
FinishBreakpoint should be kept as-is, IMO, or we can rewrite the
message like "exception thrown not caught".
> diff --git a/gdb/testsuite/gdb.server/ext-restart.exp b/gdb/testsuite/gdb.server/ext-restart.exp
> index b21c7c5..035686c 100644
> --- a/gdb/testsuite/gdb.server/ext-restart.exp
> +++ b/gdb/testsuite/gdb.server/ext-restart.exp
> @@ -60,6 +60,6 @@ with_test_prefix "restart" {
> }
> }
>
> -gdb_test "kill" "" "kill" "Kill the program being debugged.*" "y"
> +gdb_test "kill" "" "kill" "kill the program being debugged.*" "y"
>
Likewise.
> --- a/gdb/testsuite/gdb.server/ext-wrapper.exp
> +++ b/gdb/testsuite/gdb.server/ext-wrapper.exp
> @@ -65,6 +65,6 @@ with_test_prefix "restart" {
> gdb_test "print d" "\\$${decimal} = ${hex} \"1\".*"
> }
>
> -gdb_test "kill" "" "kill" "Kill the program being debugged.*" "y"
> +gdb_test "kill" "" "kill" "kill the program being debugged.*" "y"
>
> gdb_test "detach" "Detaching from.*"
> } else {
> - gdb_test "kill" "" "kill process" "Kill the program being debugged.*y or n. $" "y"
> + gdb_test "kill" "" "kill process" "kill the program being debugged.*y or n. $" "y"
> }
>
> gdb_test_no_output "set breakpoint always-inserted off"
> diff --git a/gdb/testsuite/gdb.threads/kill.exp b/gdb/testsuite/gdb.threads/kill.exp
> index a6d8382..bc00914 100644
> --- a/gdb/testsuite/gdb.threads/kill.exp
> +++ b/gdb/testsuite/gdb.threads/kill.exp
> @@ -68,7 +68,7 @@ proc test {threaded} {
> #
> # the above would mean that the remote end crashed.
>
> - gdb_test "kill" "^y" "kill program" "Kill the program being debugged\\? \\(y or n\\) $" "y"
> + gdb_test "kill" "^y" "kill program" "kill the program being debugged\\? \\(y or n\\) $" "y"
> }
> }
>
Otherwise, patch is good to me.
--
Yao
next prev parent reply other threads:[~2016-11-27 16:48 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-25 20:54 [PATCH v2 0/8] Fix gdb's testsuite test names Luis Machado
2016-11-25 20:54 ` [PATCH v2 3/8] Fix test names starting with uppercase using gdb_test_no_output Luis Machado
2016-11-27 16:58 ` Yao Qi
2016-11-25 20:54 ` [PATCH v2 7/8] Fix test names starting with uppercase using multi-line gdb_test_multiple Luis Machado
2016-11-30 20:47 ` Pedro Alves
2016-11-30 21:05 ` Luis Machado
2016-11-30 22:14 ` Pedro Alves
2016-11-25 20:54 ` [PATCH v2 6/8] Fix test names starting with uppercase using multi-line gdb_test_no_output Luis Machado
2016-11-30 20:12 ` Pedro Alves
2016-11-25 20:55 ` [PATCH v2 5/8] Fix test names starting with uppercase using multi-line gdb_test/mi_gdb_test Luis Machado
2016-11-30 20:11 ` Pedro Alves
2016-11-30 20:25 ` Luis Machado
2016-11-25 20:55 ` [PATCH v2 8/8] Fixup testcases outputting own name as a test name and standardize failed compilation messages Luis Machado
2016-12-01 11:08 ` Pedro Alves
2016-12-01 13:15 ` Luis Machado
2016-12-01 17:20 ` [PATCH v3 " Luis Machado
2016-12-01 17:35 ` Pedro Alves
2016-12-01 20:52 ` Luis Machado
2016-11-25 20:55 ` [PATCH v2 2/8] Fix test names starting with uppercase using gdb_test on a single line Luis Machado
2016-11-27 16:48 ` Yao Qi [this message]
2016-11-28 18:13 ` [PATCH v3 " Luis Machado
2016-11-25 20:55 ` [PATCH v2 1/8] Fix test names starting with uppercase output by basic functions Luis Machado
2016-11-29 21:37 ` Yao Qi
2016-11-25 20:55 ` [PATCH v2 4/8] Fix test names starting with uppercase using gdb_test_multiple Luis Machado
2016-11-27 17:10 ` Yao Qi
2016-11-28 16:06 ` Luis Machado
2016-11-29 14:49 ` Yao Qi
2016-11-29 15:07 ` Luis Machado
2016-11-29 20:55 ` Yao Qi
2016-11-29 21:14 ` Luis Machado
2016-11-30 19:56 ` Pedro Alves
2016-11-28 18:57 ` [PATCH v2 0/8] Fix gdb's testsuite test names Sergio Durigan Junior
2016-11-28 19:00 ` Luis Machado
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=20161127164749.4sdm7r47f7nbe6se@localhost \
--to=qiyaoltc@gmail.com \
--cc=gdb-patches@sourceware.org \
--cc=lgustavo@codesourcery.com \
/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