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: | Review of extension of JavaProjectConstant | ||
---|---|---|---|
Product: | java | Reporter: | Tomas Zezula <tzezula> |
Component: | Project | Assignee: | Tomas Zezula <tzezula> |
Status: | VERIFIED FIXED | ||
Severity: | blocker | Keywords: | API_REVIEW_FAST |
Priority: | P3 | ||
Version: | 5.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | |||
Bug Blocks: | 57733 | ||
Attachments: | Diff |
Description
Tomas Zezula
2005-04-20 16:15:42 UTC
Created attachment 21777 [details]
Diff
I agree. Looks OK to me. I presume the real fix would involve changes to java/freeform and/or java/j2seproject to use the new constant. Yes, the diff of a complete fix (non API part) is attached to issue #57733 Knowing there is a test for the new behaviour makes me feel more confident, the change will be useful and correct. Is see changes in a test, but why it does not refer to the new constant? The test used hardcoded values. I've fixed it to use the constants from JPC. Checking in project/src/org/netbeans/api/java/project/JavaProjectConstants.java; /cvs/java/project/src/org/netbeans/api/java/project/JavaProjectConstants.java,v <-- JavaProjectConstants.java new revision: 1.6; previous revision: 1.5 done Marking this issue as VERIFIED since the issue reporter == issue owner. |