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 28293 - Opening dialog for saving VCS profile is slow
Summary: Opening dialog for saving VCS profile is slow
Status: VERIFIED WONTFIX
Alias: None
Product: obsolete
Classification: Unclassified
Component: vcsgeneric (show other bugs)
Version: 3.x
Hardware: PC Linux
: P3 blocker (vote)
Assignee: Martin Entlicher
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2002-10-25 22:17 UTC by _ pkuzel
Modified: 2003-06-10 16:41 UTC (History)
1 user (show)

See Also:
Issue Type: TASK
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 _ pkuzel 2002-10-25 22:17:26 UTC 
It takes on PIII@600 1,5-3s.

..vcs.advanced.variables.VariableIO uses DOM's
getElementsByTagName(). This approach can be
replaced by very simple SAX handler.
Comment 1 Martin Entlicher 2002-10-29 15:04:58 UTC 
You mean VariableIO.getConfigurationLabel(Document) method?
Is there a better way how to get the first element?

It would be perhaps better to reuse
VariableIO.getConfigurationLabelAndOS() method, that uses SAX.
Comment 2 _ pkuzel 2002-10-29 15:17:40 UTC 
Node.getFirstChild may help. However you must guarantee that
dom.Document is not be garbage collected between several usages.
XMLDataObject weak references it and parses it on demand again and again.

LabelContentHandler uses minimal in-memory model so it is the best
solution (fast).
Comment 3 Martin Entlicher 2002-10-29 15:31:46 UTC 
O.K. I'll use LabelContentHandler.
Thanks.
Comment 4 Tomas Pavek 2003-06-10 15:25:51 UTC 
Is this issue still relevant in new projects (NB 4.0)?
Comment 5 Jiri Kovalsky 2003-06-10 16:26:33 UTC 
I don't think so .. :-)
Comment 6 Tomas Pavek 2003-06-10 16:36:52 UTC 
So let's close it, just reminding there is still issue 27741.
Comment 7 Jiri Kovalsky 2003-06-10 16:41:29 UTC 
Okay, agreed.