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 105085 - AttrImpl.textNode static field memory leak
Summary: AttrImpl.textNode static field memory leak
Status: VERIFIED FIXED
Alias: None
Product: obsolete
Classification: Unclassified
Component: visualweb (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: Peter Zavadsky
URL:
Keywords: PERFORMANCE
Depends on:
Blocks: 105078
  Show dependency tree
 
Reported: 2007-05-30 00:09 UTC by Quy Nguyen
Modified: 2007-07-13 21:29 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 Quy Nguyen 2007-05-30 00:09:55 UTC 
The org.apache.xerces.dom.AttrImpl class has a static field 'textNode' that
appears to cause a designer memory leak.  The source of this problem is detailed
here in an apache bug report (http://issues.apache.org/jira/browse/XERCESJ-1248).

Even though this is not under our control, a workaround or some way to mitigate
the issue might be possible.
Comment 1 Peter Zavadsky 2007-05-31 00:22:16 UTC 
Trying to hack that problem in the subclass. If that works I'll put it in, but
it might not work over the other releases. The real fix needs to happen in
Xerces impl (or to use different impl).
Comment 2 Peter Zavadsky 2007-05-31 00:24:08 UTC 
Fixed. It needs to be checked whether it works fine (memory leak), and whether
it doesn't harm performance otherwise (the speed).

Checking in
visualweb/designer/markup/src/org/netbeans/modules/visualweb/designer/markup/AbstractRaveDocument.java;
/cvs/visualweb/designer/markup/src/org/netbeans/modules/visualweb/designer/markup/AbstractRaveDocument.java,v
 <--  AbstractRaveDocument.java
new revision: 1.10; previous revision: 1.9
done
Comment 3 Quy Nguyen 2007-06-12 23:20:53 UTC 
Verified that the memory leak no longer occurs on 070612.
Comment 4 _ krystyna 2007-07-13 21:29:04 UTC 
Looks like Quy verified.