From aca075d50b493de7850f6b61770da4ff80dac6b8 Mon Sep 17 00:00:00 2001
From: Oleg Krasilnikov <oleg.krasilnikov@intel.com>
Date: Fri, 7 Sep 2007 13:27:52 +0000
Subject: [PATCH] Bug #201612 : CDT does not use the first configuration in xml
 as default

---
 .../ui/wizards/CDTConfigWizardPage.java       | 13 ++-
 .../managedbuilder/ui/wizards/CfgHolder.java  | 91 +++++++++++++++++++
 .../ui/wizards/MBSWizardHandler.java          |  1 +
 .../ui/wizards/STDWizardHandler.java          |  3 +-
 4 files changed, 104 insertions(+), 4 deletions(-)

diff --git a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/CDTConfigWizardPage.java b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/CDTConfigWizardPage.java
index 5c7e18b321f..65bc3fc30df 100644
--- a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/CDTConfigWizardPage.java
+++ b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/CDTConfigWizardPage.java
@@ -15,6 +15,7 @@ import java.util.Set;
 import java.util.TreeSet;
 
 import org.eclipse.cdt.managedbuilder.core.BuildException;
+import org.eclipse.cdt.managedbuilder.core.IConfiguration;
 import org.eclipse.cdt.managedbuilder.core.IProjectType;
 import org.eclipse.cdt.managedbuilder.core.IToolChain;
 import org.eclipse.cdt.managedbuilder.core.ManagedBuildManager;
@@ -360,10 +361,16 @@ public class CDTConfigWizardPage extends WizardPage {
 		Set x = new TreeSet();			
 		Set y = new TreeSet();			
 		for (int i=0; i<n; i++) {
+			if (tcs[i] == null) // --- NO TOOLCHAIN ---
+				continue;       // has no custom pages. 
 			x.add(tcs[i]);
-			try { // Any element may be null. Do nothing in this case. 
-				y.add(tcs[i].getParent().getProjectType().getId());
-			} catch (NullPointerException e) {}
+
+			IConfiguration cfg = tcs[i].getParent();
+			if (cfg == null)
+				continue;
+			IProjectType pt = cfg.getProjectType();
+			if (pt != null)
+				y.add(pt.getId());
 		}
 		MBSCustomPageManager.addPageProperty(MBSCustomPageManager.PAGE_ID, MBSCustomPageManager.TOOLCHAIN, x);
 		if (ptIsNull) {
diff --git a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/CfgHolder.java b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/CfgHolder.java
index 7a35928f302..18c514d5e06 100644
--- a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/CfgHolder.java
+++ b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/CfgHolder.java
@@ -13,6 +13,7 @@ package org.eclipse.cdt.managedbuilder.ui.wizards;
 import java.util.ArrayList;
 
 import org.eclipse.cdt.managedbuilder.core.IConfiguration;
+import org.eclipse.cdt.managedbuilder.core.IProjectType;
 import org.eclipse.cdt.managedbuilder.core.IToolChain;
 import org.eclipse.cdt.managedbuilder.core.ManagedBuildManager;
 import org.eclipse.cdt.ui.newui.UIMessages;
@@ -52,6 +53,13 @@ public class CfgHolder {
 		return (cfg.isSupported());
 	}
 	
+	/**
+	 * Checks whether names are unique
+	 * 
+	 * @param its
+	 * @return
+	 */
+	
     public static boolean hasDoubles(CfgHolder[] its) {
    		for (int i=0; i<its.length; i++) {
        		String s = its[i].name;
@@ -64,6 +72,13 @@ public class CfgHolder {
    		return false;
     }
 
+    /**
+     * Creates array of holders on a basis
+     * of configurations array.
+     * 
+     * @param cfgs
+     * @return
+     */
     public static CfgHolder[] cfgs2items(IConfiguration[] cfgs) {
     	CfgHolder[] its = new CfgHolder[cfgs.length];
     	for (int i=0; i<cfgs.length; i++) {
@@ -72,6 +87,15 @@ public class CfgHolder {
     	return its;
     }
  
+    /**
+     * Makes configuration's names unique.
+     * Adds either version number or toolchain name. 
+     * If it does not help, simply adds index.
+     * 
+     * @param its - list of items.
+     * @return the same list with unique names. 
+     */
+    
     public static CfgHolder[] unique(CfgHolder[] its) {
     	// if names are not unique, add version name
     	if (hasDoubles(its)) {
@@ -106,6 +130,65 @@ public class CfgHolder {
     	return its;
     }
     
+    /**
+     * Returns corresponding project type
+     * obtained either from configuration
+     * (if any) or from toolchain. 
+     * 
+     * @return projectType
+     */
+    
+    public IProjectType getProjectType() {
+    	if (cfg != null)
+    		return cfg.getProjectType();
+    	if (tc != null && tc.getParent() != null)
+    		return tc.getParent().getProjectType();
+    	return null;
+    }
+
+    /**
+     * Reorders selected configurations in "physical" order.
+     * Although toolchains are displayed in alphabetical 
+     * order in Wizard, it's required to create corresponding
+     * configurations in the same order as they are listed
+     * in xml file, inside of single project type.   
+     * 
+     * @param its - items in initial order.
+     * @return - items in "physical" order.
+     */
+    
+    public static CfgHolder[] reorder(CfgHolder[] its) {
+    	ArrayList ls = new ArrayList(its.length);
+    	boolean found = true;
+    	while (found) {
+			found = false;
+    		for (int i=0; i<its.length; i++) {
+    			if (its[i] == null) 
+    				continue;
+    			found = true;
+    			IProjectType pt = its[i].getProjectType();
+    			if (pt == null) {
+					ls.add(its[i]);
+					its[i] = null;
+    				continue;
+    			}
+    			IConfiguration[] cfs = pt.getConfigurations();
+    			for (int j=0; j<cfs.length; j++) {
+    				for (int k=0; k<its.length; k++) {
+    					if (its[k] == null) 
+    						continue;
+    					if (cfs[j].equals(its[k].getTcCfg())) {
+    						ls.add(its[k]);
+    						its[k] = null;
+    					}
+    				}
+    			}
+    		}
+    	}
+    	return (CfgHolder[])ls.toArray(new CfgHolder[ls.size()]); 
+    }
+    
+    
     /*
      * Note that null configurations are ignored !
      */
@@ -115,6 +198,14 @@ public class CfgHolder {
     		if (its[i].cfg != null) lst.add(its[i].cfg);
     	return (IConfiguration[])lst.toArray(new IConfiguration[lst.size()]);
     }
+    
+    public IConfiguration getTcCfg() {
+    	if (tc != null)
+    		return tc.getParent();
+    	return null;
+    }
+    
+    
 	public Object getConfiguration() { return cfg; }
 	public String getName() { return name; }
 	public Object getToolChain() { return tc; }
diff --git a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/MBSWizardHandler.java b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/MBSWizardHandler.java
index c38f7fdd663..17e836db54c 100644
--- a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/MBSWizardHandler.java
+++ b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/MBSWizardHandler.java
@@ -419,6 +419,7 @@ public class MBSWizardHandler extends CWizardHandler {
 		info.setManagedProject(mProj);
 
 		cfgs = CfgHolder.unique(cfgs);
+		cfgs = CfgHolder.reorder(cfgs);
 		
 		ICConfigurationDescription cfgDebug = null;
 		ICConfigurationDescription cfgFirst = null;
diff --git a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/STDWizardHandler.java b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/STDWizardHandler.java
index 6ae907f4e69..c9c5e0a7747 100644
--- a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/STDWizardHandler.java
+++ b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/wizards/STDWizardHandler.java
@@ -56,7 +56,8 @@ public class STDWizardHandler extends MBSWizardHandler {
 		info.setManagedProject(mProj);
 
 		cfgs = CfgHolder.unique(fConfigPage.getCfgItems(defaults));
-
+		cfgs = CfgHolder.reorder(cfgs);
+			
 		for (int i=0; i<cfgs.length; i++) {
 			String s = (cfgs[i].getToolChain() == null) ? "0" : ((ToolChain)(cfgs[i].getToolChain())).getId();  //$NON-NLS-1$
 			Configuration cfg = new Configuration(mProj, (ToolChain)cfgs[i].getToolChain(), ManagedBuildManager.calculateChildId(s, null), cfgs[i].getName());