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.
| Summary: | Decrease the number of classes in debugger | ||
|---|---|---|---|
| Product: | debugger | Reporter: | Petr Nejedly <pnejedly> |
| Component: | Code | Assignee: | Marian Petras <mpetras> |
| Status: | VERIFIED FIXED | ||
| Severity: | blocker | Keywords: | PERFORMANCE |
| Priority: | P1 | ||
| Version: | 3.x | ||
| Hardware: | Sun | ||
| OS: | Solaris | ||
| Issue Type: | TASK | Exception Reporter: | |
| Attachments: | A patch against debuggercore removing 11 unneeded classes from it. | ||
Created attachment 1702 [details]
A patch against debuggercore removing 11 unneeded classes from it.
Finished in all three debugger modules (debugger core, JPDA debugger, tools debugger). Well done, thanks. It seems it would be benefical to also add some more strightforward way of providing PropertyEditors without subclassing one either Property of PropertyEditor... |
There are a lot of anonymous Node.Property _subclasses_ in the debugger (as well as in other modules) which can be mostly (in ~90%, maybe more) replaced by using PropertySupport.Reflection _instances_ I'll attach a patch as an example how it could be achieved. Please use the same or similar technique whenever possible. grep "PropertySupport.Read" be your friend. Apply the same measure to other debugger modules. PS:I'm not sure about src/org/netbeans/modules/debugger/support/nodes/VariableNode.java, Watch.PROP_AS_TEXT, because it is also catching catch (DebuggerException e) {}