Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-debug-dev] Bug in MIBreakInsert

Yes, no need for bugzilla or patch. Thank you.

----- Original Message ----- 
From: "Thomas Fletcher" <ThomasF@xxxxxxx>
To: "CDT Debug developers list" <cdt-debug-dev@xxxxxxxxxxx>
Sent: Friday, April 14, 2006 10:38 AM
Subject: RE: [cdt-debug-dev] Bug in MIBreakInsert


> 
> 
> > -----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
> _______________________________________________
> cdt-debug-dev mailing list
> cdt-debug-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-debug-dev


Back to the top