[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
RE: [cdt-debug-dev] BreakpointManager.java is broken
|
Yes, patches have two main requirements.
1) They need to be in bugzilla so that we can keep tabs on them for IP checking purposes at Eclipse.
2) They have to be good. There are many criteria for this. But you need to consider that there are many users of the CDT out there using it in many ways. Patches that don't consider this are unacceptable. And, unfortunately, we've had many patches that fit into that category.
Doug.
-----Original Message-----
From: cdt-debug-dev-bounces@xxxxxxxxxxx [mailto:cdt-debug-dev-bounces@xxxxxxxxxxx] On Behalf Of Elena Laskavaia
Sent: Wednesday, April 02, 2008 10:08 AM
To: CDT Debug developers list
Subject: Re: [cdt-debug-dev] BreakpointManager.java is broken
If you remove this breakpoints on static functions would not work.
Btw how can you create function breakpoint from UI which does not have proper file?
In any case you cannot attach path to this e-mail it has to be bug report.
Øyvind Harboe wrote:
>
> The attached patch disables broken code in BreakpointManager.java
>
> I don't know what the fix is, but the file name can be the binary e.g.
> foo.elf, so the syntax break foo.c:xxx can't be blindly used.
>
> I think it is best to disable this code until it can be implemented
> correctly.
>
>
> Objections?
>
> --
> Øyvind Harboe
> http://www.zylin.com - eCos ARM & FPGA developer kit
>
_______________________________________________
cdt-debug-dev mailing list
cdt-debug-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-debug-dev
