From 7d16f0f551b33f39244a0fb12c73f067e1bf4622 Mon Sep 17 00:00:00 2001 From: Ben Elliston Date: Mon, 21 Mar 2016 11:32:31 +1100 Subject: [PATCH] * baseboards/mcore-moto-sim.exp: Remove old FIXME comment. * baseboards/mmixware-sim.exp: Likewise. * config/sim.exp: Likewise. * runtest.exp: Likewise. --- ChangeLog | 7 +++++++ baseboards/mcore-moto-sim.exp | 3 --- baseboards/mmixware-sim.exp | 3 --- config/sim.exp | 3 --- runtest.exp | 1 - 5 files changed, 7 insertions(+), 10 deletions(-) diff --git a/ChangeLog b/ChangeLog index 1b8b095..4ad5b02 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,12 @@ 2016-03-21 Ben Elliston + * baseboards/mcore-moto-sim.exp: Remove old FIXME comment. + * baseboards/mmixware-sim.exp: Likewise. + * config/sim.exp: Likewise. + * runtest.exp: Likewise. + +2016-03-21 Ben Elliston + * lib/dg.exp (dg-get-options): Clean up regular expression and remove FIXME comment about it. diff --git a/baseboards/mcore-moto-sim.exp b/baseboards/mcore-moto-sim.exp index 0259ec0..e0321b0 100644 --- a/baseboards/mcore-moto-sim.exp +++ b/baseboards/mcore-moto-sim.exp @@ -109,9 +109,6 @@ proc sim_load { dest prog args } { set status $status2 } - # FIXME: Do we need to examine $status? - # Yes, we do--what if the simulator itself gets an error and coredumps? - verbose "Return status was: $status" 2 if { $status == 0 } { set result "pass" diff --git a/baseboards/mmixware-sim.exp b/baseboards/mmixware-sim.exp index 4cc52c1..1d6cfbd 100644 --- a/baseboards/mmixware-sim.exp +++ b/baseboards/mmixware-sim.exp @@ -73,6 +73,3 @@ set_board_info ldscript "" # Return codes aren't supported by the simulator. set_board_info needs_status_wrapper 1 - -# FIXME: Some gdb board variables are usually here. I currently have -# no clue what to set them to, so I won't. diff --git a/config/sim.exp b/config/sim.exp index 87bd403..9b196e3 100644 --- a/config/sim.exp +++ b/config/sim.exp @@ -113,9 +113,6 @@ proc sim_load { dest prog args } { set status $status2 } - # FIXME: Do we need to examine $status? - # Yes, we do--what if the simulator itself gets an error and coredumps? - verbose "Return status was: $status" 2 if { $status == 0 } { set result "pass" diff --git a/runtest.exp b/runtest.exp index ccd2a4d..e53afdf 100644 --- a/runtest.exp +++ b/runtest.exp @@ -1694,7 +1694,6 @@ foreach current_target $target_list { lappend restore "$var" } # Handle "CFLAGS=$CFLAGS foo". - # FIXME: Do we need to `catch' this? eval set $var \[string range \"$varval\" [expr {$tmp + 1}] end\] verbose "$var is now [eval concat \$$var]" unset tmp var -- 2.7.4