Remove 'eval' from gdb_breakpoint
authorTom Tromey <tromey@adacore.com>
Wed, 22 Feb 2023 22:21:45 +0000 (15:21 -0700)
committerTom Tromey <tromey@adacore.com>
Thu, 23 Feb 2023 13:49:20 +0000 (06:49 -0700)
Now that Tcl has the {*} operator, we can remove the use of eval from
gdb_breakpoint.  Tested on x86-64 Fedora 36.

gdb/testsuite/lib/gdb.exp

index 6864dac3ac91f9c9753caf7a437c1aedbfa37ab4..66a5d31d9ad539eb33fc31322d90db9b656972c1 100644 (file)
@@ -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
     }