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 96456 - Statically held resources in wsdl model
Summary: Statically held resources in wsdl model
Status: RESOLVED FIXED
Alias: None
Product: xml
Classification: Unclassified
Component: WSDL Model (show other bugs)
Version: 6.x
Hardware: All All
: P4 blocker (vote)
Assignee: Ritesh Adval
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2007-02-23 15:41 UTC by _ rkubacki
Modified: 2007-03-03 17:02 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

Attachments
diff (27.46 KB, application/octet-stream)
2007-03-02 02:25 UTC, Ritesh Adval 		Details
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 _ rkubacki 2007-02-23 15:41:04 UTC 
Various resources like images or resourcebundles are assigned to static
variables in this module. It reduces possibility to effectively manage memory -
once these classes are initialized the data cannot be GC'ed. It is better to use
existing API that support caching to allow fast access but also make it possible
to reclaim memory.

org/netbeans/modules/xml/wsdl/model/extensions/soap/validation/SOAPComponentValidator.class
in
/opt/devel/nb/nb_all/nbbuild/netbeans/ide8/modules/org-netbeans-modules-xml-wsdl-model.jar
Comment 1 Ritesh Adval 2007-02-28 01:21:38 UTC 
which existing API you are talking about which supports caching?

Is it NBBundle?
Comment 2 Petr Nejedly 2007-02-28 07:58:48 UTC 
Both NbBundle and IconManager (Utilities.loadImage) have smart caches built in.
Comment 3 Ritesh Adval 2007-02-28 22:04:18 UTC 
Ok we will fix this.
Comment 4 Ritesh Adval 2007-03-02 02:24:38 UTC 
This is now fixed in nb6.0
Comment 5 Ritesh Adval 2007-03-02 02:25:29 UTC 
Created attachment 39083 [details]
diff
Comment 6 Shivanand Kini 2007-03-02 02:35:17 UTC 
reviewed the code changes. looks good.
Comment 7 _ rkubacki 2007-03-02 07:01:35 UTC 
however I apreciate that you take care about this I do not see strong enough
reason to backport this to 5.5.1. It is in highresistance mode so P4 bugs are
not fixed there at this moment.

So once again thanks for fixing and keep things easy and limit number of changes
in branches that are close to complete freeze.
Comment 8 Ritesh Adval 2007-03-03 03:10:35 UTC 
Ok sounds good. This fixed in nb6.0 only.
Comment 9 _ rkubacki 2007-03-03 17:02:09 UTC 
updating TM