Fixes issue #65. Always ignore empty strings for appendages.

Author: kwin

com.googlecode.cppcheclipse.core/src/com/googlecode/cppcheclipse/core/Appendages.java

@@ -10,6 +10,7 @@
 
 import com.google.common.base.Joiner;
 import com.google.common.base.Splitter;
+import com.google.common.base.Strings;
 import com.googlecode.cppcheclipse.core.utils.SerializeHelper;
 
 public class Appendages implements TableModel<File> {
@@ -26,13 +27,18 @@ public Appendages(IPreferenceStore preferenceStore) {
 	}
 
 	private void load() {
-		Iterable<String> values = Splitter.on(DELIMITER).split(preferenceStore
-				.getString(IPreferenceConstants.P_APPENDAGES));
+		Iterable<String> values = Splitter
+				.on(DELIMITER)
+				.omitEmptyStrings()
+				.split(preferenceStore
+						.getString(IPreferenceConstants.P_APPENDAGES));
 		for (String file : values) {
 			try {
 				files.add((File) SerializeHelper.fromString(file));
 			} catch (Exception e) {
-				CppcheclipsePlugin.logWarning("Error reading filename of appendage", e);
+				CppcheclipsePlugin.logWarning(
+						"Error reading filename of appendages. Stored appendage file '"
+								+ file + "'", e);
 			}
 		}
 	}
@@ -43,29 +49,30 @@ public void save() throws IOException {
 		for (File file : files) {
 			values.add(SerializeHelper.toString(file));
 		}
-		preferenceStore.setValue(IPreferenceConstants.P_APPENDAGES, Joiner.on(DELIMITER).join(values));
+		preferenceStore.setValue(IPreferenceConstants.P_APPENDAGES,
+				Joiner.on(DELIMITER).join(values));
 	}
-	
+
 	public void removeAll() {
 		files.clear();
 	}
-	
+
 	public void add(File file) {
 		files.add(file);
 	}
-	
+
 	public void remove(File file) {
 		files.remove(file);
 	}
 
 	public Iterator<File> iterator() {
 		return files.iterator();
 	}
-	
+
 	public boolean isEmpty() {
 		return files.isEmpty();
 	}
-	
+
 	public File[] toArray() {
 		return files.toArray(new File[files.size()]);
 	}