Merge pull request #4473 from bclothier/FixComSafeDisposal

Fix up the Dispose methods

Author: retailcoder

Rubberduck.VBEEditor/ComManagement/ComSafeBase.cs

@@ -28,7 +28,10 @@ protected int GetComWrapperObjectHashCode(ISafeComWrapper comWrapper)
         public void Dispose()
         {
             Dispose(true);
+        }
 
+        protected virtual void Dispose(bool disposing)
+        {
 #if DEBUG
             if (_disposed)
             {
@@ -67,8 +70,6 @@ public void Dispose()
 #endif
         }
 
-        protected abstract void Dispose(bool disposing);
-
 #if DEBUG
         private struct TraceData
         {

Rubberduck.VBEEditor/ComManagement/StrongComSafe.cs

@@ -57,6 +57,7 @@ protected override void Dispose(bool disposing)
             }
 
             _disposed = true;
+            base.Dispose(disposing);
 
             foreach (var comWrapper in _comWrapperCache.Keys)
             {

Rubberduck.VBEEditor/ComManagement/WeakComSafe.cs

@@ -61,6 +61,7 @@ protected override void Dispose(bool disposing)
             }
 
             _disposed = true;
+            base.Dispose(disposing);
 
             foreach (var weakReference in _comWrapperCache.Values)
             {