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

Summary:	Statically held resources in wsdl model
Product:	xml	Reporter:	_ rkubacki <rkubacki>
Component:	WSDL Model	Assignee:	Ritesh Adval <radval>
Status:	RESOLVED FIXED
Severity:	blocker	CC:	issues
Priority:	P4	Keywords:	PERFORMANCE
Version:	6.x
Hardware:	All
OS:	All
Issue Type:	DEFECT	Exception Reporter:
Attachments:	diff

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