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 - Remove gdb classes instantiated at startup
Summary: Remove gdb classes instantiated at startup
Status: RESOLVED WONTFIX
Alias: None
Product: cnd
Classification: Unclassified
Component: Debugger (show other bugs)
Version: 6.x
Hardware: All All
: P3 blocker (vote)
Assignee: _ gordonp
URL:
Keywords: PERFORMANCE
Depends on: 127518
Blocks: 127408
  Show dependency tree
 
Reported: 2008-02-15 16:22 UTC by _ gordonp
Modified: 2009-11-02 11:17 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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