If no objections, let's follow these excellent guidelines form Mylyn. Please note especially the @Overides directive. I think that answers our issue regarding how to make use of that keyword.
Code- For error handling use
StatusHandler and pass an IStatus object. Note that some of the other Mylyn code uses the convenience methods on StatusHandler , but we are phasing those out because they do not pass the plug-in ID that the exception originated from. - Use
WorkbenchJob for running jobs that should only run when the workbench is active. Not doing this can cause errors on workbench shutdown (e.g. bug 178409). - When using
String.toLowerCase() , use String.toLowerCase(Locale.ENGLISH) to ensure locale safety (see bug 168652). - Do not use
@Override annotations on implementing methods, only on overriding methods. Doing so violates Java 5 (bug 173171). - Use DateFormat with extra caution. It is not thread-safe and should not be saved to fields in classes that can be used from multiple threads (UI, asynchonous execution, or jobs).
- For the sake of multi-monitor setups, use
getMonitor() instead of getDisplay() when you want to position a UI element on a specific coordinate of the screen. - To improve the speed of decorators use setUseHashlookup(true) on all structured viewers.
______________________________ Miles T. Parker President and Chief Software Architect Metascape, LLC Project Lead Eclipse Agent Modeling Project
tel: 509-643-4441 skype: milestravisparker ______________________________
|