aboutsummaryrefslogtreecommitdiff
path: root/gdb/testsuite
diff options
context:
space:
mode:
authorTom Tromey <tromey@adacore.com>2023-02-22 15:21:45 -0700
committerTom Tromey <tromey@adacore.com>2023-02-23 06:49:20 -0700
commite98a23bfb3088fb129540d7fd26f3205e2de6f2a (patch)
tree129bbf6a23ddd0b201eb38d0e7a1ce5720e2a866 /gdb/testsuite
parent0373a76af6b8ef949b9ca8aaa6325fbde0948603 (diff)
downloadgdb-e98a23bfb3088fb129540d7fd26f3205e2de6f2a.zip
gdb-e98a23bfb3088fb129540d7fd26f3205e2de6f2a.tar.gz
gdb-e98a23bfb3088fb129540d7fd26f3205e2de6f2a.tar.bz2
Remove 'eval' from gdb_breakpoint
Now that Tcl has the {*} operator, we can remove the use of eval from gdb_breakpoint. Tested on x86-64 Fedora 36.
Diffstat (limited to 'gdb/testsuite')
-rw-r--r--gdb/testsuite/lib/gdb.exp7
1 files changed, 1 insertions, 6 deletions
diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 6864dac..66a5d31 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -704,12 +704,7 @@ proc runto { linespec args } {
set test_name "runto: run to $linespec"
- # We need to use eval here to pass our varargs args to gdb_breakpoint
- # which is also a varargs function.
- # But we also have to be careful because $linespec may have multiple
- # elements, and we don't want Tcl to move the remaining elements after
- # the first to $args. That is why $linespec is wrapped in {}.
- if ![eval gdb_breakpoint {$linespec} $args] {
+ if {![gdb_breakpoint $linespec {*}$args]} {
return 0
}