Anonymous avatar Anonymous committed bc988b4

encapsulating null check bugs fixes in SosController.types as a property

Comments (0)

Files changed (1)

     {
         private ListViewItem[] processes;
 
-        private ListViewItem[] types;
+        /// <summary>
+				/// don't use me, use SosController.types instead
+				/// </summary>
+				private ListViewItem[] _types;
 
-        private string cdbPath;
+				private ListViewItem[] types
+				{
+				  get { return _types ?? new ListViewItem[]{}; }
+				  set { _types = value; }
+				}
+
+    	private string cdbPath;
 
         private List<string> commandHistory = new List<string>();
 
 
         public ListViewItem[] FilterTypes(string typeName)
         {
-        	var typestofilterover = this.types ?? new ListViewItem[] {};
         	Func<ListViewItem, bool> filteringPredicate =
         		(item) =>
         		item.SubItems[3].Text.IndexOf(typeName, StringComparison.OrdinalIgnoreCase) != -1;
-          return typestofilterover.Where(filteringPredicate).ToArray();
+          return this.types.Where(filteringPredicate).ToArray();
         }
 
         public ObjectInfo GetInstanceDumpDetails(string instanceAddress)
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.