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 46606 - Many SAX parsers held by winsys during IDE session
Summary: Many SAX parsers held by winsys during IDE session
Status: CLOSED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Window System (show other bugs)
Version: 4.x
Hardware: PC All
: P4 blocker (vote)
Assignee: Milos Kleint
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2004-07-27 12:51 UTC by _ rkubacki
Modified: 2008-12-22 18:50 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
patch for winsys module (9.04 KB, patch)
2004-07-27 12:53 UTC, _ rkubacki
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description _ rkubacki 2004-07-27 12:51:31 UTC
Memory profiling shows that we have more than 50
org.apache.xerces.parsers.SAXParser instances
allocated on heap while the app is running. They
are useless and can be released. PropertyHandlers
in o.n.c.windows.persistence hold them together
with some configuration and bunch of other objects
belonging to Xerces. 

If I get it right they are held because the system
uses these handlers for saving of winsys state
during the shutdown. I will attach a patch that
saves the memory and creates new instances of
PropertyHandler-s for saving. The better approach
can be to split loading and saving. I have not
measured impact on startup time. It should be
pretty small as we will collect some object
instead of promoting them.

It should give us some tens of kB (43 instance of
SAXParser, XML11Configuration, a lot of QNames,
some string instances).
Comment 1 Milos Kleint 2004-07-27 12:52:40 UTC
reassigning
Comment 2 _ rkubacki 2004-07-27 12:53:08 UTC
Created attachment 16484 [details]
patch for winsys module
Comment 3 _ rkubacki 2004-07-27 17:08:45 UTC
Also char arrays containing whole file content for winsys files
vanished with the patch. 
Comment 4 Milos Kleint 2004-07-28 12:49:10 UTC
applied the patch.
Comment 5 _ rkubacki 2004-08-05 10:01:23 UTC
x