Merge pull request #133 from Dmitry-Me/removeEmptyLinesBeforeBuildingTheSet

Remove empty lines before building the set

Author: VioletGiraffe

CPPCheckPlugin/SuppressionsSettingsUI/ItemsListEditor.xaml.cs

@@ -33,7 +33,11 @@ public string Label
 		public HashSet<string> Items
 		{
 			set { ItemsListText.Text = string.Join("\n", value); }
-			get { return new HashSet<string>(ItemsListText.Text.Split(new string[] { "\r\n", "\n" }, StringSplitOptions.None)); }
+			get {
+				var text = ItemsListText.Text;
+				var splitted = text.Split(new string[] { "\r\n", "\n" }, StringSplitOptions.RemoveEmptyEntries);
+				return new HashSet<string>(splitted);
+			}
 		}
 	}
 }

CPPCheckPlugin/SuppressionsSettingsUI/SuppressionsSettings.xaml.cs

@@ -31,11 +31,8 @@ private void SaveClick(object sender, RoutedEventArgs e)
 			SuppressionsInfo suppressionsInfo = new SuppressionsInfo();
 
 			suppressionsInfo.SuppressionLines = CppcheckLines.Items;
-			while (suppressionsInfo.SuppressionLines.Remove("")) { /* nothing */ }
 			suppressionsInfo.SkippedFilesMask = FilesLines.Items;
-			while (suppressionsInfo.SkippedFilesMask.Remove("")) { /* nothing */ }
 			suppressionsInfo.SkippedIncludesMask = IncludesLines.Items;
-			while (suppressionsInfo.SkippedIncludesMask.Remove("")) { /* nothing */ }
 
 			suppressionsInfo.SaveToFile(suppressionsFilePath);