From 87a07e829c627060134bbca1eb34ed564612e360 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christoph=20D=C3=B6bber?= <doebber@in.tum.de>
Date: Thu, 5 Jan 2012 15:38:27 +0000
Subject: [PATCH] major beautifications refs 181

---
 .../ui/internal/views/MarkerViewPart.java     | 26 ++++++++++---------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/internal/views/MarkerViewPart.java b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/internal/views/MarkerViewPart.java
index 765d8f4df..cef28e976 100644
--- a/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/internal/views/MarkerViewPart.java
+++ b/org.fortiss.tooling.kernel.ui/trunk/src/org/fortiss/tooling/kernel/ui/internal/views/MarkerViewPart.java
@@ -32,12 +32,12 @@ import org.eclipse.emf.ecore.EObject;
 import org.eclipse.jface.action.Action;
 import org.eclipse.jface.action.IToolBarManager;
 import org.eclipse.jface.action.Separator;
+import org.eclipse.jface.resource.ImageDescriptor;
 import org.eclipse.jface.viewers.CellLabelProvider;
 import org.eclipse.jface.viewers.ITreeContentProvider;
 import org.eclipse.jface.viewers.Viewer;
 import org.eclipse.jface.viewers.ViewerCell;
 import org.eclipse.swt.SWT;
-import org.eclipse.swt.graphics.Image;
 import org.eclipse.swt.layout.FillLayout;
 import org.eclipse.swt.widgets.Composite;
 import org.eclipse.ui.part.ViewPart;
@@ -138,7 +138,8 @@ public class MarkerViewPart extends ViewPart {
 			public void update(ViewerCell cell) {
 				if (cell.getElement() instanceof ESeverity) {
 					cell.setText(((ESeverity) cell.getElement()).toString());
-					cell.setImage(getImageFor((ESeverity) cell.getElement()));
+					cell.setImage(getImageFor((ESeverity) cell.getElement())
+							.createImage());
 				} else {
 					cell.setText("");
 				}
@@ -244,6 +245,7 @@ public class MarkerViewPart extends ViewPart {
 			this.severity = severity;
 			setToolTipText(tooltip);
 			setChecked(toggleState.get(severity));
+			setImageDescriptor(getImageFor(severity));
 		}
 
 		/** {@inheritDoc} */
@@ -256,23 +258,23 @@ public class MarkerViewPart extends ViewPart {
 	}
 
 	/** Returns the icon appropriate for the severity. */
-	public Image getImageFor(ESeverity severity) {
+	public ImageDescriptor getImageFor(ESeverity severity) {
 		switch (severity) {
 		case FATAL:
-			return ToolingKernelUIActivator.getImageDescriptor(
-					"icons/fatal.gif").createImage();
+			return ToolingKernelUIActivator
+					.getImageDescriptor("icons/fatal.gif");
 		case ERROR:
-			return ToolingKernelUIActivator.getImageDescriptor(
-					"icons/error.gif").createImage();
+			return ToolingKernelUIActivator
+					.getImageDescriptor("icons/error.gif");
 		case WARNING:
-			return ToolingKernelUIActivator.getImageDescriptor(
-					"icons/warning.gif").createImage();
+			return ToolingKernelUIActivator
+					.getImageDescriptor("icons/warning.gif");
 		case INFO:
 			return ToolingKernelUIActivator
-					.getImageDescriptor("icons/info.gif").createImage();
+					.getImageDescriptor("icons/info.gif");
 		case DEBUG:
-			return ToolingKernelUIActivator.getImageDescriptor(
-					"icons/debug.gif").createImage();
+			return ToolingKernelUIActivator
+					.getImageDescriptor("icons/debug.gif");
 
 		}
 
-- 
GitLab