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: | Add ModuleInfo.getClassLoader() | ||
---|---|---|---|
Product: | platform | Reporter: | Jesse Glick <jglick> |
Component: | Module System | Assignee: | Jesse Glick <jglick> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | CC: | apireviews |
Priority: | P2 | Keywords: | API, API_REVIEW_FAST |
Version: | 3.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | ENHANCEMENT | Exception Reporter: | |
Bug Depends on: | |||
Bug Blocks: | 38306 | ||
Attachments: | Proposed patch |
Description
Jesse Glick
2003-12-26 01:36:25 UTC
See issue #38306 for branch information. Actually it seems o.n.c.m.M.gCLOrNull is unnecessary - no one was trying to call M.getClassLoader on a disabled module anyway. Created attachment 12675 [details]
Proposed patch
Requesting fast-track review. Simple compatible change. The method has long existed internally in core; this just exposes it in the Open APIs. No change needed to arch descs. Correction: it seems autoupdate does in fact override ModuleInfo for its own internal purposes (though it does not expose the instances externally). So ModuleInfo.getClassLoader should not be abstract but should instead throw UnsupportedOperationException in the default impl - this will permit autoupdate to remain unchanged. Will make this correction in the branch. Planned for commit to trunk during the day (US time) on Wednesday. committed * Up-To-Date 1.33 core/manifest.mf committed * Up-To-Date 1.50 core/src/org/netbeans/core/modules/Module.java committed * Up-To-Date 1.129 openide/openide-spec-vers.properties committed * Up-To-Date 1.182 openide/api/doc/changes/apichanges.xml committed * Up-To-Date 1.11 openide/src/org/openide/modules/ModuleInfo.java |