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 28445 - SAX wizard panels hurt performance
Summary: SAX wizard panels hurt performance
Status: VERIFIED FIXED
Alias: None
Product: xml
Classification: Unclassified
Component: Code (show other bugs)
Version: 3.x
Hardware: All All
: P2 blocker (vote)
Assignee: _ pkuzel
URL:
Keywords: PERFORMANCE
Depends on:
Blocks: 27795
  Show dependency tree
 
Reported: 2002-11-04 14:36 UTC by David Simonek
Modified: 2003-07-10 10:56 UTC (History)
0 users

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 David Simonek 2002-11-04 14:36:34 UTC 
WizardDescriptor.Panel implementations
org\netbeans\modules\xml\core\wizard\AbstractPanel.java
org\netbeans\modules\xml\tools\generator\SAXGeneratorAbstractPanel.java

provide itself as the result of getComponent().
This hurts performance and can cause a clash when
Component.isValid() is overridden.
Please use a separate component class, see details
at
http://performance.netbeans.org/howto/dialogs/wizard-panels.html.

Easy to fix, P2 justification - performance is top
priority.
Comment 1 _ pkuzel 2002-11-05 09:16:31 UTC 
I'll fix it soon...
Comment 2 _ pkuzel 2002-11-05 17:48:39 UTC 
core/wizard used in New from template isValid() clash fixed. It was
not P9E problem because it uses custom WizardIterator that lazy
initializes wizard panels.
Comment 3 David Simonek 2002-11-05 17:59:13 UTC 
Agreed, but only halfway. Still it's better to create expensive UI
panel in getComponent, so that system can decide when it's right time
to do it - see detailed description section in howto.
Comment 4 _ pkuzel 2002-11-22 17:49:55 UTC 
Tools wizard panels fixed.
Comment 5 Martin Schovanek 2003-03-14 12:38:41 UTC 
VERIFIED
Comment 6 _ pkuzel 2003-06-16 13:35:38 UTC 
Removed #34223 blocker. already in prj40_prototype build.