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: | Remove the Hacks class from the Project UI module | ||
---|---|---|---|
Product: | projects | Reporter: | Petr Hrebejk <phrebejk> |
Component: | Generic Infrastructure | Assignee: | Milos Kleint <mkleint> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | CC: | jglick, jrechtacek |
Priority: | P4 | Keywords: | ARCH |
Version: | 4.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: | |
Bug Depends on: | |||
Bug Blocks: | 41535, 49990 |
Description
Petr Hrebejk
2004-04-08 13:29:02 UTC
I already got rid of some stuff, but more remains. Also there should not remain any overrides or masks in the projects/projectui layer. Provisionally marking as a defect to make sure it is considered in the D timeframe. What remains in Hacks is adding the project specific new Action into the FolderLoader. (I doubt that this will disapear in D) And the update of the MainWindow title. This is probably up to you Jesse. The layer looks pretty clean. We are only hiding the Folder template which is probably OK. Well probably can't do much about the rest. The remaining hacks slows down startup. Re window title: Can we enhance for instance StatusDisplayer with a method to set app title text? That way we can avoid reflection to NbTopManager that takes signifact time in ModuleRestored. Would it be feasible? No API changes now please. Can be considered for E. StatusDisplayer is the wrong place, anyway; should just be a public method in o.o.w.WindowManager, I suppose. Re. overhead for using reflection to access NbTopManager.WindowSystem: surely this cannot be significant! We are using one Class.forName and one Class.getMethod. Perhaps you are seeing the effect of some reflection system being initialized? Or if it really consumes a measurable amount of time to load one Method in the current JRE, we should be filing a JRE performance bug. Almost all hacks are removed already. The only ramining is the Main window title which is waiting for support from winsys I think. Downgrading to P4. Updating window title no longer uses reflection and is here to stay. jrechtacek added some reflective call to TemplateWizard.reload(DataObject) which indicates an API defect somewhere, so he can evaluate that. closing the issue as fixed as I understand from the comments everything is done. The TemplateWizard.reload(DataObject) hack should probably still be reevaluated. |