From a3961ab19f5b1aa3848038338646fd0b21998bc1 Mon Sep 17 00:00:00 2001
From: Oleg Krasilnikov <oleg.krasilnikov@intel.com>
Date: Thu, 10 May 2007 09:13:06 +0000
Subject: [PATCH] Bug 186306: possible tool's NULL name.

---
 .../managedbuilder/ui/properties/BuildListComparator.java   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/BuildListComparator.java b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/BuildListComparator.java
index 7a45da43d84..f821fe3b4d9 100644
--- a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/BuildListComparator.java
+++ b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/BuildListComparator.java
@@ -31,7 +31,11 @@ public class BuildListComparator extends CDTListComparator {
 		if (a instanceof ITool) {
 			ITool c1 = (ITool)a;
 			ITool c2 = (ITool)b;
-			return c1.getName().compareToIgnoreCase(c2.getName());
+			String s1 = c1.getName();
+			if (s1 == null) s1 = "";  //$NON-NLS-1$
+			String s2 = c2.getName();
+			if (s2 == null) s2 = "";  //$NON-NLS-1$ 
+			return s2.compareToIgnoreCase(s2);
 		}
 		if (a instanceof IBuildPropertyValue) {
 			IBuildPropertyValue c1 = (IBuildPropertyValue)a;