[RFA] breakpoint.c: Avoid double freeing in breakpoint_re_set_one

[Andrew Cagney]

Yep.

Few tweaks.

> # IDT/SIM apparently doesn't have enough file descriptors to allow the
> # problem checked by this test to occur.
> if [istarget "mips-idt-*"] {
>     return 0;
> }

is this true?  If it is it looks more like a bug.  Other workarounds may 
be in the same situtation.

Just check that the braces are indented:

	gdb_expect {
	    -re ... {
	    }
	}

thanks,
Andrew