sort the content of .autobuild and autobuild.cfg

For version control sorting is a good idea as it will reduce unwanted
differences due to differences in sequence in these generated files

Author: jan

io.sloeber.autoBuild/src/io/sloeber/autoBuild/helpers/api/KeyValueTree.java

@@ -3,9 +3,11 @@
 import static io.sloeber.autoBuild.helpers.api.AutoBuildConstants.*;
 
 import java.util.AbstractMap.SimpleEntry;
+import java.util.ArrayList;
 import java.io.File;
 import java.io.IOException;
 import java.util.LinkedHashMap;
+import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
 import org.apache.commons.io.FileUtils;
@@ -95,10 +97,10 @@ public String toString() {
 		return getKey() + EQUAL + getValue();
 	}
 
-	private String toStringLine(KeyValueTree theRoot) {
+	private void toStringLine(KeyValueTree theRoot,List<String> output) {
 		if ((null == getValue()) || (this == theRoot)) {
 			// leaves without a value should not be logged
-			return EMPTY_STRING;
+			return ;
 		}
 		String seperator = EMPTY_STRING;
 		String ret = EQUAL + getValue();
@@ -108,7 +110,7 @@ private String toStringLine(KeyValueTree theRoot) {
 			current = current.getParent();
 			seperator = DOT;
 		}
-		return ret + NEWLINE;
+		output.add(ret);
 	}
 
 	public Map<String, String> toKeyValues(boolean addParents) {
@@ -123,17 +125,21 @@ public Map<String, String> toKeyValues(String prefix) {
 		return toKeyValues(prefix, this);
 	}
 
-	public String dump() {
-		return dump(this);
+	public String  dump() {
+		ArrayList<String> ret=new ArrayList<>();
+		 dump(this, ret);
+		 return String.join(NEWLINE,ret);
 	}
 
-	private String dump(KeyValueTree theRoot) {
+	public void  dump(List<String> output) {
+		 dump(this, output);
+	}
 
-		String stringRepresentation = toStringLine(theRoot);
+	private void dump(KeyValueTree theRoot,List<String> output) {
+		toStringLine(theRoot,output);
 		for (KeyValueTree node : getChildren().values()) {
-			stringRepresentation += node.dump(theRoot);
+			node.dump(theRoot,output);
 		}
-		return stringRepresentation;
 	}
 
 	private Map<String, String> toKeyValues(String prefix, KeyValueTree theRoot) {

io.sloeber.autoBuild/src/io/sloeber/autoBuild/integration/AutoBuildConfigurationDescriptionProvider.java

@@ -18,6 +18,8 @@
 import java.io.File;
 import java.io.IOException;
 import java.nio.charset.Charset;
+import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Set;
 import org.apache.commons.io.FileUtils;
 import org.eclipse.cdt.core.settings.model.ICConfigurationDescription;
@@ -141,7 +143,11 @@ public CConfigurationData applyConfiguration(ICConfigurationDescription cfgDescr
 
 			// save the project file if needed
 			boolean needsWriting = true;
-			String configText = keyValuePairs.dump();
+
+			ArrayList<String> contentLines=new ArrayList<>();
+			keyValuePairs.dump(contentLines);
+			contentLines.sort(null);
+			String configText = String.join(NEWLINE,contentLines);
 			if (projectFile.exists()) {
 				String curConfigsText = FileUtils.readFileToString(projectFile, AUTOBUILD_CONFIG_FILE_CHARSET);
 				needsWriting = !curConfigsText.equals(configText);
@@ -167,7 +173,11 @@ public CConfigurationData applyConfiguration(ICConfigurationDescription cfgDescr
 
 			// save the team file if needed
 			needsWriting = true;
-			String teamText = keyValuePairs.dump();
+			ArrayList<String> teamContentLines=new ArrayList<>();
+			keyValuePairs.dump(teamContentLines);
+			teamContentLines.sort(null);
+			String teamText = String.join(NEWLINE,teamContentLines);
+
 			if (teamText.length() < 2) {
 				teamFile.delete(true, monitor);
 				needsWriting = false;
@@ -188,6 +198,7 @@ public CConfigurationData applyConfiguration(ICConfigurationDescription cfgDescr
 		return baseData;
 	}
 
+
 	@Override
 	public CConfigurationData createConfiguration(ICConfigurationDescription cfgDescription,
 			ICConfigurationDescription baseCfgDescription, CConfigurationData base, boolean clone,