[PATCH] testsuite/config/sim.exp:gdb_load fix return values

[PATCH] testsuite/config/sim.exp:gdb_load fix return values

[Richard Earnshaw]

[-- Attachment #1: Type: text/plain, Size: 257 bytes --]


This patch fixes the return values of the gdb_load implementation in 
sim.exp.  With this patch the testsuite is begining to look somewhat more 
sane again for an arm-elf cross.

OK?

	* config/sim.exp (gdb_load): Return 0 on success, -1 on failure.

R.



[-- Attachment #2: sim.patch --]
[-- Type: text/x-patch , Size: 769 bytes --]

Index: sim.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/config/sim.exp,v
retrieving revision 1.2
diff -p -p -r1.2 sim.exp
*** sim.exp	8 Feb 2004 03:56:14 -0000	1.2
--- sim.exp	27 Nov 2004 12:59:44 -0000
*************** proc gdb_load { arg } {
*** 71,77 ****
  	    }
  	    set timeout 30
  	    verbose "Timeout is now $timeout seconds" 2
! 	    return 1
  	}
  	-re "$gdb_prompt $"     {
  	    if $verbose>1 then {
--- 71,77 ----
  	    }
  	    set timeout 30
  	    verbose "Timeout is now $timeout seconds" 2
! 	    return 0
  	}
  	-re "$gdb_prompt $"     {
  	    if $verbose>1 then {
*************** proc gdb_load { arg } {
*** 84,87 ****
--- 84,88 ----
  	    }
  	}
      }
+     return -1
  }

Re: [PATCH] testsuite/config/sim.exp:gdb_load fix return values

[Andrew Cagney]

Richard Earnshaw wrote:
> This patch fixes the return values of the gdb_load implementation in 
> sim.exp.  With this patch the testsuite is begining to look somewhat more 
> sane again for an arm-elf cross.
> 
> OK?
> 
> 	* config/sim.exp (gdb_load): Return 0 on success, -1 on failure.

M'kay (for michael).

Andrew

> ------------------------------------------------------------------------
> 
> Index: sim.exp
> ===================================================================
> RCS file: /cvs/src/src/gdb/testsuite/config/sim.exp,v
> retrieving revision 1.2
> diff -p -p -r1.2 sim.exp
> *** sim.exp	8 Feb 2004 03:56:14 -0000	1.2
> --- sim.exp	27 Nov 2004 12:59:44 -0000
> *************** proc gdb_load { arg } {
> *** 71,77 ****
>   	    }
>   	    set timeout 30
>   	    verbose "Timeout is now $timeout seconds" 2
> ! 	    return 1
>   	}
>   	-re "$gdb_prompt $"     {
>   	    if $verbose>1 then {
> --- 71,77 ----
>   	    }
>   	    set timeout 30
>   	    verbose "Timeout is now $timeout seconds" 2
> ! 	    return 0
>   	}
>   	-re "$gdb_prompt $"     {
>   	    if $verbose>1 then {
> *************** proc gdb_load { arg } {
> *** 84,87 ****
> --- 84,88 ----
>   	    }
>   	}
>       }
> +     return -1
>   }