get_integer_valueof: Don't output value in test name
authorSimon Marchi <simon.marchi@ericsson.com>
Tue, 10 Oct 2017 19:10:25 +0000 (15:10 -0400)
committerSimon Marchi <simon.marchi@ericsson.com>
Tue, 10 Oct 2017 19:10:25 +0000 (15:10 -0400)
The get_integer_valueof outputs the value it has read as part of the
test name.  This causes test names to vary from run to run, and adds
some noise when diffing test results.  e.g.:

-PASS: gdb.threads/process-dies-while-detaching.exp: multi-process: continue: killed outside: get integer valueof "mypid" (28770)
+PASS: gdb.threads/process-dies-while-detaching.exp: multi-process: continue: killed outside: get integer valueof "mypid" (32238)

This patch removes that, since it's probably not very useful.

gdb/testsuite/ChangeLog:

* lib/gdb.exp (get_integer_valueof): Don't output read value in test name.

gdb/testsuite/ChangeLog
gdb/testsuite/lib/gdb.exp

index 4ad0c02992a6f783bb793c43738376029b87ae41..dd0645c80fc018779391b71d469d1cac44bdb3df 100644 (file)
@@ -1,3 +1,7 @@
+2017-10-10  Simon Marchi  <simon.marchi@ericsson.com>
+
+       * lib/gdb.exp (get_integer_valueof): Don't output read value in test name.
+
 2017-10-10  Pedro Alves <palves@redhat.com>
 
        * gdb.mi/mi-pthreads.exp (check_mi_thread_command_set): Don't
index 48fec2f56b91a62a518c966d1bc7bb354588ff47..9d8296a109eb18f637f43d2c95186e0fc7bfbe95 100644 (file)
@@ -5634,7 +5634,7 @@ proc get_integer_valueof { exp default {test ""} } {
     gdb_test_multiple "print /d ${exp}" "$test" {
        -re "\\$\[0-9\]* = (\[-\]*\[0-9\]*).*$gdb_prompt $" {
            set val $expect_out(1,string)
-           pass "$test ($val)"
+           pass "$test"
        }
        timeout {
            fail "$test (timeout)"