RE: [cdt-debug-dev] Bug in MIBreakInsert

[Thomas Fletcher <ThomasF@xxxxxxx>]

> -----Original Message-----
> From: cdt-debug-dev-bounces@xxxxxxxxxxx 
> [mailto:cdt-debug-dev-bounces@xxxxxxxxxxx] On Behalf Of Mark Mitchell
> Sent: April 14, 2006 12:50 AM
> To: CDT Debug developers list
> Subject: [cdt-debug-dev] Bug in MIBreakInsert
> 
> MIBreakInsert incorrectly assumes that a breakpoint cannot be 
> both a temporary breakpoint and a hardware breakpoint:
> 
> In particular, in:
> 
> 		if (isTemporary) {
> 			opts[i] = "-t"; //$NON-NLS-1$
> 			i++;
> 		} else if (isHardware) {
> 			opts[i] = "-h"; //$NON-NLS-1$
> 			i++;
> 		}
> 
> the "else" should be removed.
> 
> How should I submit a patch to get this fixed?

Patch to this list is fine, or a PR with the patch attached will work
too.  In this case it is pretty simple so Mikhail may pick it up for
free.

Thomas