Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* Re: A Patch/fix
       [not found] <5.0.2.1.2.20020920151310.01988d30@mail.netburner.com>
@ 2002-09-24 20:10 ` Jim Blandy
  2002-09-24 21:09   ` Andrew Cagney
  0 siblings, 1 reply; 3+ messages in thread
From: Jim Blandy @ 2002-09-24 20:10 UTC (permalink / raw)
  To: Paul Breed; +Cc: gdb-patches

Paul Breed <Paul@Netburner.com> writes:

> I'm not familiar with the procedure for submitting a patch,
> but I have a simple one line fix that is  a bug.
> 
> in gdb/main.c line 384
> 
> baud_rate=i;
> }
> <----------------------insert a "break;" here
> case ''l':
> 
> 
> Without the break when you set the baud rate you also set the time out to some
> ungodly long amount.

Committed as obvious:

2002-09-24  Jim Blandy  <jimb@redhat.com>

	Fix from Paul Breed:
	* main.c (captured_main): Add a `break' after the case for 'b'.

Index: gdb/main.c
===================================================================
RCS file: /cvs/src/src/gdb/main.c,v
retrieving revision 1.18
diff -c -r1.18 main.c
*** gdb/main.c	25 Jun 2002 05:35:16 -0000	1.18
--- gdb/main.c	25 Sep 2002 03:07:30 -0000
***************
*** 397,402 ****
--- 397,403 ----
  	      else
  		baud_rate = i;
  	    }
+             break;
  	  case 'l':
  	    {
  	      int i;



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: A Patch/fix
  2002-09-24 20:10 ` A Patch/fix Jim Blandy
@ 2002-09-24 21:09   ` Andrew Cagney
  2002-09-24 21:27     ` Jim Blandy
  0 siblings, 1 reply; 3+ messages in thread
From: Andrew Cagney @ 2002-09-24 21:09 UTC (permalink / raw)
  To: Jim Blandy; +Cc: Paul Breed, gdb-patches

Don't forget 5.3.
Andrew

> Paul Breed <Paul@Netburner.com> writes:
> 
> 
>> I'm not familiar with the procedure for submitting a patch,
>> but I have a simple one line fix that is  a bug.
>> 
>> in gdb/main.c line 384
>> 
>> baud_rate=i;
>> }
>> <----------------------insert a "break;" here
>> case ''l':
>> 
>> 
>> Without the break when you set the baud rate you also set the time out to some
>> ungodly long amount.
> 
> 
> Committed as obvious:
> 
> 2002-09-24  Jim Blandy  <jimb@redhat.com>
> 
> 	Fix from Paul Breed:
> 	* main.c (captured_main): Add a `break' after the case for 'b'.
> 
> Index: gdb/main.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/main.c,v
> retrieving revision 1.18
> diff -c -r1.18 main.c
> *** gdb/main.c	25 Jun 2002 05:35:16 -0000	1.18
> --- gdb/main.c	25 Sep 2002 03:07:30 -0000
> ***************
> *** 397,402 ****
> --- 397,403 ----
>   	      else
>   		baud_rate = i;
>   	    }
> +             break;
>   	  case 'l':
>   	    {
>   	      int i;
> 
> 
> 



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: A Patch/fix
  2002-09-24 21:09   ` Andrew Cagney
@ 2002-09-24 21:27     ` Jim Blandy
  0 siblings, 0 replies; 3+ messages in thread
From: Jim Blandy @ 2002-09-24 21:27 UTC (permalink / raw)
  To: Andrew Cagney; +Cc: Paul Breed, gdb-patches


Andrew Cagney <ac131313@redhat.com> writes:
> Don't forget 5.3.

Done.  Thanks for the reminder.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2002-09-25  4:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <5.0.2.1.2.20020920151310.01988d30@mail.netburner.com>
2002-09-24 20:10 ` A Patch/fix Jim Blandy
2002-09-24 21:09   ` Andrew Cagney
2002-09-24 21:27     ` Jim Blandy

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox