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 71353 - Avoid unused BeanInfos for TAX elements
Summary: Avoid unused BeanInfos for TAX elements
Status: RESOLVED FIXED
Alias: None
Product: xml
Classification: Unclassified
Component: TAX (show other bugs)
Version: 5.x
Hardware: All All
: P3 blocker (vote)
Assignee: issues@xml
URL:
Keywords: PERFORMANCE
Depends on:
Blocks:
 
Reported: 2006-01-13 10:35 UTC by _ rkubacki
Modified: 2008-02-29 22:42 UTC (History)
4 users (show)

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 _ rkubacki 2006-01-13 10:35:08 UTC 
xml/tax module registers an entry to bean info search path and contains many
BeanInfo classes for xml/tax elements. These were probably used to create nodes
for XML tree editor but this no longer exists so we can remove them as well. It
will reduce size of the module and improve speed of bean introspection for all
other places in the IDE.
Comment 1 Sherold Dev 2006-07-27 15:33:35 UTC 
tm -> 5.5
Comment 2 Samaresh Panda 2007-09-28 00:41:35 UTC 
Marek, do you know if it is valid anymore? Well, is xml/tax being used by anyone? I do see this module in
cluster.properties but the project neither has any public packages nor friends. Sounds like this module is not used
anymore, I'll remove it from cluster.properties and see if the build works.
Comment 3 _ rkubacki 2007-09-28 01:26:48 UTC 
Last time when I was checking Jesse Glick said that it is used in apisupport.
Comment 4 Samaresh Panda 2008-01-22 15:35:44 UTC 
Jesse, are you still using tax?
Comment 5 Jesse Glick 2008-01-23 21:19:19 UTC 
Yes, apisupport/project still relies on TAX for parsing and regenerating XML layer files.
Comment 6 Samaresh Panda 2008-01-24 22:07:58 UTC 
Jesse, Do you have any objections to removing these classes?
Comment 7 Jesse Glick 2008-01-25 00:05:10 UTC 
Go ahead, I don't think anyone cares about the BI for these classes.
Comment 8 Samaresh Panda 2008-02-01 17:57:35 UTC 
Have removed them: http://hg.netbeans.org/main?cmd=changeset;node=d01a1930ab2c. I do see a lot of unused classes in
org.netbeans.modules.xml.tax.beans.customizer as well. Pl. confirm if it's ok to delete them as well?
Comment 9 Jesse Glick 2008-02-01 18:10:28 UTC 
I don't know anything about that package so I assume it's probably safe to delete.