VBComponents:

mansellan · mansellan · commit a2971ea93c60 · 2017-10-06T16:12:10.000+01:00
- Added nullchecks to ImportSourceFile() and RemoveSafely()
diff --git a/Rubberduck.VBEEditor/SafeComWrappers/VBA/VBComponents.cs b/Rubberduck.VBEEditor/SafeComWrappers/VBA/VBComponents.cs
@@ -101,7 +101,7 @@ public void ImportSourceFile(string path)
         {
             var ext = Path.GetExtension(path);
             var name = Path.GetFileNameWithoutExtension(path);
-            if (!File.Exists(path))
+            if (!File.Exists(path) || this[name] == null)
             {
                 return;
             }
@@ -152,6 +152,9 @@ public void ImportSourceFile(string path)
 
         public void RemoveSafely(IVBComponent component)
         {
+            if (component == null)
+                { return; }
+
             switch (component.Type)
             {
                 case ComponentType.ClassModule:

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ public void ImportSourceFile(string path)`
`101`	`101`	`{`
`102`	`102`	`var ext = Path.GetExtension(path);`
`103`	`103`	`var name = Path.GetFileNameWithoutExtension(path);`
`104`		`- if (!File.Exists(path))`
	`104`	`+ if (!File.Exists(path) \|\| this[name] == null)`
`105`	`105`	`{`
`106`	`106`	`return;`
`107`	`107`	`}`
`@@ -152,6 +152,9 @@ public void ImportSourceFile(string path)`
`152`	`152`
`153`	`153`	`public void RemoveSafely(IVBComponent component)`
`154`	`154`	`{`
	`155`	`+ if (component == null)`
	`156`	`+ { return; }`
	`157`	`+`
`155`	`158`	`switch (component.Type)`
`156`	`159`	`{`
`157`	`160`	`case ComponentType.ClassModule:`