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.
Summary: | Patch to be used for measuring classpath scan time | ||
---|---|---|---|
Product: | java | Reporter: | Marian Mirilovic <mmirilovic> |
Component: | Unsupported | Assignee: | Pavel Flaska <pflaska> |
Status: | CLOSED FIXED | ||
Severity: | blocker | CC: | apireviews, issues, jtulach |
Priority: | P2 | Keywords: | API_REVIEW_FAST |
Version: | 4.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | TASK | Exception Reporter: | |
Attachments: | here is the patch |
Description
Marian Mirilovic
2005-02-08 15:19:49 UTC
Pavel, what is the status of this issue ? Thanks and sorry ;) From my point of view, I do not have anything against it as it is only code which runs with private property. I have only one question. Do we really need to use reflection? (Sorry for late answer.) I will add your property to arch-javacore.xml. > From my point of view, I do not have anything against it as it is only > code which runs with private property. I have only one question. Do we > really need to use reflection? Do you know about any other solution ? > (Sorry for late answer.) > I will add your property to arch-javacore.xml. That's fine , thanks. Yarda, a little bird told me that you are the boss responsible for approving reflection usage in our code base, so please could you look at this ? I would like to have this code in trunk before NB4.1 Beta branching, Thanks in advance. I was speaking with Yarda and he agreed with this change. Pavel or Martin, I am not sure what has to be done for Arch, so if I can help you let me know what, where and ... I am applying for fast review, because we would like to measure scan classpath as soon as possible, at least we need data for Beta , and Beta is going to be branched on 15th of February ;( Created attachment 20311 [details]
here is the patch
Just document in arch.xml that there is such strange contract (api of property and reflection). OK, we are going to merge this within the next 24 hours. Checking in arch/arch-javacore.xml; /cvs/java/javacore/arch/arch-javacore.xml,v <-- arch-javacore.xml new revision: 1.17; previous revision: 1.16 done Processing log script arguments... More commits to come... Checking in src/org/netbeans/modules/javacore/JMManager.java; /cvs/java/javacore/src/org/netbeans/modules/javacore/JMManager.java,v <-- JMManager.java new revision: 1.93; previous revision: 1.92 done Pavel, thanks a lot - verified. Reorganization of java component |