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: | static API method UI.invokeAfterScanFinished | ||
---|---|---|---|
Product: | java | Reporter: | Jan Becicka <jbecicka> |
Component: | Refactoring | Assignee: | Jan Becicka <jbecicka> |
Status: | RESOLVED INVALID | ||
Severity: | normal | CC: | jglick, tzezula |
Priority: | P3 | Keywords: | API_REVIEW, API_REVIEW_FAST |
Version: | 7.2 | ||
Hardware: | PC | ||
OS: | Mac OS X | ||
Issue Type: | TASK | Exception Reporter: | |
Attachments: | Proposed patch |
Description
Jan Becicka
2011-12-09 14:04:06 UTC
Created attachment 114003 [details]
Proposed patch
[JG01] Is there some reason this is not in, say, java.sourceui or parsing.api? It does not seem to use anything in refactoring.java (other than a suspicious use of bundle keys from outside the package). [JG01] the only request is from refactoring afaik. this is the reason JG01 - that is what I guessed, but if there is no inherent reason why an API cannot be placed in a more basic module, and you can foresee possible uses from non-refactoring-related code, then it seems strange to put it where you did. JL01 (JG01 cont'ed): What is the relation to java.sourceui's ScanDialog.runWhenScanFinished? The javadoc seems very similar, if not the same, and the impl. is also somewhat similar. ScanDialog.runWhenScanFinished is used on a couple of places in NetBeans proper: http://lahoda.info/index/usages/search?path=netbeans-indexing&signatures=METHOD:org.netbeans.api.java.source.ui.ScanDialog:runWhenScanFinished:%28Ljava/lang/Runnable;Ljava/lang/String;%29Z The java source ui module already provides such a method: org.netbeans.api.java.source.ui.ScanDialog.runWhenScanFinished looks like the method was already copied to proper place, but was not removed from refactoring. Thanks for review. |