Merge pull request #2387 from retailcoder/next

Vbe.Interop.Forms dependency fix, and don't iterate references+components in protected projects when releasing

Author: retailcoder

RetailCoder.VBE/Rubberduck.csproj

@@ -247,6 +247,7 @@
       <SpecificVersion>False</SpecificVersion>
       <EmbedInteropTypes>False</EmbedInteropTypes>
       <HintPath>..\libs\Microsoft.Vbe.Interop.Forms.dll</HintPath>
+      <Private>True</Private>
     </Reference>
     <Reference Include="Ninject.Extensions.Conventions">
       <HintPath>..\packages\ninject.extensions.conventions.3.2.0.0\lib\net45-full\Ninject.Extensions.Conventions.dll</HintPath>

Rubberduck.VBEEditor/Rubberduck.VBEditor.csproj

@@ -105,6 +105,7 @@
       <SpecificVersion>False</SpecificVersion>
       <EmbedInteropTypes>False</EmbedInteropTypes>
       <HintPath>..\libs\Microsoft.Vbe.Interop.Forms.dll</HintPath>
+      <Private>True</Private>
     </Reference>
     <Reference Include="office">
       <HintPath>..\libs\Office.dll</HintPath>

Rubberduck.VBEEditor/SafeComWrappers/VBA/VBProject.cs

@@ -117,8 +117,11 @@ public override void Release(bool final = false)
         {
             if (!IsWrappingNullReference)
             {
-                References.Release();
-                VBComponents.Release();
+                if (Protection == ProjectProtection.Unprotected)
+                {
+                    References.Release();
+                    VBComponents.Release();
+                }
                 base.Release(final);
             }
         }