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 13116 - Decrease the number of classes in debugger
Summary: Decrease the number of classes in debugger
Status: VERIFIED FIXED
Alias: None
Product: debugger
Classification: Unclassified
Component: Code (show other bugs)
Version: 3.x
Hardware: Sun Solaris
: P1 blocker (vote)
Assignee: Marian Petras
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2001-06-22 16:13 UTC by Petr Nejedly
Modified: 2001-09-05 12:03 UTC (History)
0 users

See Also:
Issue Type: TASK
Exception Reporter:

Attachments
A patch against debuggercore removing 11 unneeded classes from it. (14.63 KB, patch)
2001-07-20 20:30 UTC, Petr Nejedly 		Details | Diff
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 Petr Nejedly 2001-06-22 16:13:13 UTC 
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) {}
Comment 1 Petr Nejedly 2001-06-22 16:15:15 UTC 
Created attachment 1702 [details]
A patch against debuggercore removing 11 unneeded classes from it.
Comment 2 Marian Petras 2001-09-03 21:36:17 UTC 
Finished in all three debugger modules (debugger core, JPDA debugger,
tools debugger).
Comment 3 Petr Nejedly 2001-09-05 12:03:30 UTC 
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...