Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] CDT ToggleBreakpointsTargetFactory enablement

Hi Pawel,

Your proposed fix for scope will work well together with disabling CDT factories. Will you be making the change you have suggested in Platform Debug?

I'll file a bug for CDT and will provide a patch.


From: cdt-dev-bounces@xxxxxxxxxxx [cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Pawel Piech [pawel.piech@xxxxxxxxxxxxx]
Sent: March 15, 2011 5:03 PM
To: cdt-dev@xxxxxxxxxxx
Subject: Re: [cdt-dev] CDT ToggleBreakpointsTargetFactory enablement

I have no objections to the proposed changes, but I'd like to point out
another weakness in the current design:
The toggle breakpoint target is selected based on the factory enablement
and the IToggleBreakpointsTargetFactory.getDefaultToggleTarget() return
values.  However, once user overrides the default setting by using the
Breakpoint Types sub-menu, the user selection overrides other choices
from then on.  Would it help to change this logic so that the
user-override is reverse-able and/or applied to a narrower scope?


On 03/15/2011 12:40 PM, Chuong, Patrick wrote:
> Hi All,
> Our customers use both GDB and JTAG debugging at the same time and setting breakpoint between debug sessions are cumbersome, the toggle breakpoint type selection need to be switch to match the breakpoint type inorder to create the corresponding breakpoint for each debug session.
> I would like to change the way CDT toggle breakpoint target factory enablement, by making them more specific to the current debug context, project type and IDeclaration element.
> For Disassembly view: the factory will be enabled when the debug session is either cdi or dsf-gdb.
> For Editor: the factory will be enabled when the editor belongs to a project that supports ICBreakpoint Type, a CProject will have a new project scope preference.
> For IDeclaration: the factory will behave similar to editor.
> Does any one see any problem with this approach or have other suggest?
> Thanks,
> Patrick
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx

cdt-dev mailing list

Back to the top