This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 127492

Summary:	Remove gdb classes instantiated at startup
Product:	cnd	Reporter:	_ gordonp <gordonp>
Component:	Debugger	Assignee:	_ gordonp <gordonp>
Status:	RESOLVED WONTFIX
Severity:	blocker	CC:	jtulach
Priority:	P3	Keywords:	PERFORMANCE
Version:	6.x
Hardware:	All
OS:	All
Issue Type:	DEFECT	Exception Reporter:
Bug Depends on:	127518
Bug Blocks:	127408

Description _ gordonp 2008-02-15 16:22:24 UTC

This is a gdb-specific version of Jarda's 127408 filed against cnd. I think it will be easier
to track if I have a separate issue...

Comment 1 _ gordonp 2008-02-15 16:24:21 UTC

Here is the initial set of gdb classes in Jarda's email. Some have already been resolved:

    > org.netbeans.modules.cnd.debugger.gdb.actions.GdbActionHandlerProvider
    > org.netbeans.modules.cnd.debugger.gdb.actions.ToggleBreakpointActionProvider
    > org.netbeans.modules.cnd.debugger.gdb.EditorContext
    > org.netbeans.modules.cnd.debugger.gdb.EditorContextBridge
    > org.netbeans.modules.cnd.debugger.gdb.EditorContextImpl
    > org.netbeans.modules.cnd.debugger.gdb.EditorContextImpl$EditorLookupListener
    > org.netbeans.modules.cnd.debugger.gdb.GdbDebuggerModule
    > org.netbeans.modules.cnd.debugger.gdb.profiles.ui.ProfileNodeProvider
    > org.netbeans.modules.cnd.debugger.gdb.profiles.ui.ProfileNodeProvider$CndProfileGeneralCustomizerNode

Comment 2 _ gordonp 2008-02-15 19:11:08 UTC

Between Thomas and I we've removed GdbActionHandlerProvider, GdbDebuggerModule,
and ProfileNodeProvider*.

I don't believe ToggleBreakpointActionProvider and EditorContext* can be removed
without a debuggercore API change. Our implementation is nearly identical to the
JPDA debugger implementation and experiments to change it broke toggle breakpoints.

I'll contact Martin and see if he has any thoughts on how TBAP can be done more
lazily. Since its the cause of the EditorContext* classes, it looks like a lazy
approach here would resolve the others, too.

Comment 3 Jaroslav Tulach 2008-02-15 19:59:23 UTC

Thanks for looking at this.

Re. the debugger core API classes: These classes are loaded for every type of debugger (bpel for example) . That very 
likely means we need a central fix. If you tell Martin, then I'll be thankful as that is going to be generally useful. 
Please keep me in loop.

Comment 4 _ gordonp 2008-02-15 21:51:21 UTC

I've sent a request to Martin asking for a debuggercore API addition to support lazy
creation of toggle breakpoint actions. The remaining gdb classes can't be removed
without that change, so I'm changing this to a P3 and making it depend on an IZ I
filed for this debuggercore API change.

Once the debuggercore changes, I can change my implementation and these classes
should get created later on.

Comment 5 _ gordonp 2008-02-28 01:06:05 UTC

Changing target milestone to future since this is blocked by 127518 and requires
an API change.

Comment 6 _ gordonp 2008-11-17 18:23:01 UTC

Can't do anything until 127492 gets fixed (its currently TBD)

Comment 7 Quality Engineering 2009-11-02 11:17:41 UTC

NetBeans.org Migration: changing resolution from LATER to WONTFIX