]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
[gdb/testsuite] get_valueof: Don't output value in test name
authorTom de Vries <tdevries@suse.de>
Wed, 31 Oct 2018 15:33:46 +0000 (16:33 +0100)
committerTom de Vries <tdevries@suse.de>
Wed, 31 Oct 2018 16:45:38 +0000 (17:45 +0100)
The get_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.guile/scm-ports.exp: buffered: get valueof "$sp" (140737488343920)
+PASS: gdb.guile/scm-ports.exp: buffered: get valueof "$sp" (140737488343968)
-PASS: gdb.guile/scm-ports.exp: unbuffered: get valueof "$sp" (140737488343920)
+PASS: gdb.guile/scm-ports.exp: unbuffered: get valueof "$sp" (140737488343968)

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

Tested on x86_64-linux.

2018-10-31  Tom de Vries  <tdevries@suse.de>

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

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

index 8fa6b0d674fe9417be38f2c6dba4b1690f111898..2f0085b6e09b35e192d4aba15f6337ab4a9eabca 100644 (file)
@@ -1,3 +1,7 @@
+2018-10-31  Tom de Vries  <tdevries@suse.de>
+
+       * lib/gdb.exp (get_valueof): Don't output read value in test name.
+
 2018-10-31  Andrew Burgess  <andrew.burgess@embecosm.com>
 
        * gdb.dwarf2/void-type.c: New file.
index b4cf80e38a0a8152f20de5d25ed81822b1ba8ffc..5a5713b1148201cd2cf55349990767466ea4e7d9 100644 (file)
@@ -5636,7 +5636,7 @@ proc get_valueof { fmt exp default {test ""} } {
     gdb_test_multiple "print${fmt} ${exp}" "$test" {
        -re "\\$\[0-9\]* = (\[^\r\n\]*)\[\r\n\]*$gdb_prompt $" {
            set val $expect_out(1,string)
-           pass "$test ($val)"
+           pass "$test"
        }
        timeout {
            fail "$test (timeout)"