Re: Patch: Little improvement to delete breakpoint command

Daniel Jacobowitz
2006-11-28T17:25:10Z

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