Re: Patch: Little improvement to delete breakpoint command

[Daniel Jacobowitz <drow@false.org>]

On Tue, Nov 28, 2006 at 06:21:39PM +0100, Markus Deuling wrote:
> Hello,
> 
> this little patch adds a break to the loop that is looking for 
> breakpoints to delete in delete_command(). I think it is only necessary 
> to find one breakpoint, there is no need to iterate further if found one.
> 
> ===================================================================
> --- breakpoint.old      2006-11-28 17:08:11.000000000 +0100
> +++ breakpoint.c        2006-11-28 17:42:15.000000000 +0100
> @@ -6960,6 +6960,7 @@
>             b->type != bp_overlay_event &&
>             b->number >= 0)
>           breaks_to_delete = 1;
> +         break;
>        }
> 
>        /* Ask user only if there are some breakpoints to delete.  */
> 

Did you test this?  I'm pretty sure you're missing braces.

-- 
Daniel Jacobowitz
CodeSourcery