Modernize remaining getDefault().getLog() log sites

Replace Plugin.getDefault().getLog() with ILog.of(Class) / ILog.get()
to drop the Activator round-trip and collapse Status construction where
applicable.

In ProjectEncodingMarkerResolutionGenerator, the class lives in
org.eclipse.ui.ide but previously logged via UIPlugin (org.eclipse.ui);
ILog.of(...) now routes the log to the containing bundle, which matches
the class location.

The workbench bundle (org.eclipse.ui.workbench) is intentionally
deferred until the org.eclipse.ui vs org.eclipse.ui.workbench plug-in
id question is resolved.

Author: vogella

bundles/org.eclipse.ui.ide/src/org/eclipse/ui/views/markers/internal/ProjectEncodingMarkerResolutionGenerator.java

@@ -25,6 +25,7 @@
 import org.eclipse.core.resources.ResourcesPlugin;
 import org.eclipse.core.resources.WorkspaceJob;
 import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.ILog;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Status;
@@ -35,15 +36,13 @@
 import org.eclipse.swt.widgets.Display;
 import org.eclipse.ui.IMarkerResolution;
 import org.eclipse.ui.IMarkerResolutionGenerator2;
-import org.eclipse.ui.internal.UIPlugin;
 import org.eclipse.ui.internal.ide.IDEWorkbenchPlugin;
 import org.eclipse.ui.views.markers.WorkbenchMarkerResolution;
 
 /**
  * Provides a resolution for warning markers on projects without an explicit
  * encoding setting.
  */
-@SuppressWarnings("restriction")
 public class ProjectEncodingMarkerResolutionGenerator implements IMarkerResolutionGenerator2 {
 
 	@Override
@@ -54,7 +53,7 @@ public IMarkerResolution[] getResolutions(IMarker marker) {
 			IMarkerResolution[] resolutions = { new ExplicitEncodingResolution(defaultCharset) };
 			return resolutions;
 		} catch (CoreException e) {
-			UIPlugin.getDefault().getLog().log(e.getStatus());
+			ILog.of(ProjectEncodingMarkerResolutionGenerator.class).log(e.getStatus());
 			return new IMarkerResolution[0];
 		}
 	}
@@ -117,7 +116,7 @@ public void run(IMarker marker) {
 					try {
 						charsetManager.setCharsetFor(project.getFullPath(), charset);
 					} catch (CoreException e) {
-						UIPlugin.getDefault().getLog().log(e.getStatus());
+						ILog.of(ProjectEncodingMarkerResolutionGenerator.class).log(e.getStatus());
 					}
 				}
 			}

bundles/org.eclipse.ui/src/org/eclipse/ui/internal/UIPreferenceInitializer.java

@@ -18,6 +18,7 @@
  *******************************************************************************/
 package org.eclipse.ui.internal;
 
+import org.eclipse.core.runtime.ILog;
 import org.eclipse.core.runtime.Platform;
 import org.eclipse.core.runtime.preferences.AbstractPreferenceInitializer;
 import org.eclipse.core.runtime.preferences.DefaultScope;
@@ -156,7 +157,7 @@ public void initializeDefaultPreferences() {
 								.getSingleton());
 			}
 		} catch (BackingStoreException e) {
-			UIPlugin.getDefault().getLog().error(e.getLocalizedMessage(), e);
+			ILog.of(UIPreferenceInitializer.class).error(e.getLocalizedMessage(), e);
 		}
 
 		rootNode

examples/org.eclipse.ui.examples.javaeditor/Template Editor Example/org/eclipse/ui/examples/templateeditor/editors/TemplateEditorUI.java

@@ -19,8 +19,7 @@
 
 import org.osgi.service.prefs.BackingStoreException;
 
-import org.eclipse.core.runtime.IStatus;
-import org.eclipse.core.runtime.Status;
+import org.eclipse.core.runtime.ILog;
 import org.eclipse.core.runtime.preferences.InstanceScope;
 
 import org.eclipse.jface.preference.IPreferenceStore;
@@ -77,7 +76,7 @@ public TemplateStore getTemplateStore() {
 			try {
 				fStore.load();
 			} catch (IOException e) {
-				JavaEditorExamplePlugin.getDefault().getLog().log(new Status(IStatus.ERROR, "org.eclipse.ui.examples.javaeditor", IStatus.OK, "", e)); //$NON-NLS-1$ //$NON-NLS-2$
+				ILog.of(TemplateEditorUI.class).error("", e); //$NON-NLS-1$
 			}
 		}
 		return fStore;

tests/org.eclipse.core.filebuffers.tests/src/org/eclipse/core/filebuffers/tests/ResourceHelper.java

@@ -18,6 +18,7 @@
 import java.io.InputStream;
 
 import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.ILog;
 import org.eclipse.core.runtime.IPath;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.core.runtime.NullProgressMonitor;
@@ -74,7 +75,7 @@ public static void delete(final IProject project, boolean deleteContent) {
 				i= MAX_RETRY;
 			} catch (CoreException x) {
 				if (i == MAX_RETRY - 1) {
-					FileBuffersTestPlugin.getDefault().getLog().log(x.getStatus());
+					ILog.of(ResourceHelper.class).log(x.getStatus());
 //					throw x;
 				}
 				try {

tests/org.eclipse.ltk.ui.refactoring.tests/src/org/eclipse/ltk/ui/refactoring/tests/RefactoringUITestPlugin.java

@@ -13,6 +13,7 @@
  *******************************************************************************/
 package org.eclipse.ltk.ui.refactoring.tests;
 
+import org.eclipse.core.runtime.ILog;
 import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Status;
 
@@ -35,7 +36,7 @@ public static String getPluginId() {
 	}
 
 	public static void log(IStatus status) {
-		getDefault().getLog().log(status);
+		ILog.get().log(status);
 	}
 
 	public static void log(Throwable e) {