Merge pull request #4378 from retailcoder/pull4238b

retailcoder · web-flow · commit 6df939d9edd3 · 2018-09-26T10:43:57.000-04:00
COM Wrapper Analyzer
diff --git a/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteCallStatementInspection.cs b/Rubberduck.CodeAnalysis/Inspections/Concrete/ObsoleteCallStatementInspection.cs
@@ -30,7 +30,8 @@ protected override IEnumerable<IInspectionResult> DoGetInspectionResults()
             foreach (var context in Listener.Contexts.Where(context => !IsIgnoringInspectionResultFor(context.ModuleName, context.Context.Start.Line)))
             {
                 string lines;
-                using (var module = State.ProjectsProvider.Component(context.ModuleName).CodeModule)
+                var component = State.ProjectsProvider.Component(context.ModuleName);
+                using (var module = component.CodeModule)
                 {
                     lines = module.GetLines(context.Context.Start.Line,
                         context.Context.Stop.Line - context.Context.Start.Line + 1);
diff --git a/Rubberduck.CodeAnalysis/QuickFixes/IgnoreOnceQuickFix.cs b/Rubberduck.CodeAnalysis/QuickFixes/IgnoreOnceQuickFix.cs
@@ -31,7 +31,9 @@ public override void Fix(IInspectionResult result)
             var annotationText = $"'@Ignore {result.Inspection.AnnotationName}";
 
             int annotationLine;
-            using (var module = _state.ProjectsProvider.Component(result.QualifiedSelection.QualifiedName).CodeModule)
+            string codeLine;
+            var component = _state.ProjectsProvider.Component(result.QualifiedSelection.QualifiedName);
+            using (var module = component.CodeModule)
             {
                 annotationLine = result.QualifiedSelection.Selection.StartLine;
                 while (annotationLine != 1 && module.GetLines(annotationLine - 1, 1).EndsWith(" _"))
diff --git a/Rubberduck.Core/Navigation/CodeExplorer/CodeExplorerComponentViewModel.cs b/Rubberduck.Core/Navigation/CodeExplorer/CodeExplorerComponentViewModel.cs
@@ -110,7 +110,8 @@ public CodeExplorerComponentViewModel(CodeExplorerItemViewModel parent, Declarat
 
         private bool ContainsBuiltinDocumentPropertiesProperty()
         {
-            using (var properties = _projectsProvider.Component(Declaration.QualifiedName.QualifiedModuleName).Properties)
+            var component = _projectsProvider.Component(Declaration.QualifiedName.QualifiedModuleName);
+            using (var properties = component.Properties)
             {
                 return properties.Any(item => item.Name == "BuiltinDocumentProperties");
             }
diff --git a/Rubberduck.Core/Navigation/RegexSearchReplace/RegexSearchReplace.cs b/Rubberduck.Core/Navigation/RegexSearchReplace/RegexSearchReplace.cs
@@ -89,7 +89,11 @@ private IEnumerable<RegexSearchResult> GetResultsFromModule(ICodeModule module,
 
         private void SetSelection(RegexSearchResult item)
         {
-            item.Module.CodePane.Selection = item.Selection;
+            var module = item.Module;
+            using (var codePane = module.CodePane)
+            {
+                codePane.Selection = item.Selection;
+            }
         }
 
         private List<RegexSearchResult> SearchSelection(string searchPattern)
diff --git a/Rubberduck.Core/UI/CodeExplorer/Commands/PrintCommand.cs b/Rubberduck.Core/UI/CodeExplorer/Commands/PrintCommand.cs
@@ -33,7 +33,8 @@ protected override bool EvaluateCanExecute(object parameter)
 
             try
             {
-                using (var codeModule = _projectsProvider.Component(node.Declaration.QualifiedName.QualifiedModuleName).CodeModule)
+                var component = _projectsProvider.Component(node.Declaration.QualifiedName.QualifiedModuleName);
+                using (var codeModule = component.CodeModule)
                 {
                     return codeModule.CountOfLines != 0;
                 }
@@ -54,7 +55,8 @@ protected override void OnExecute(object parameter)
                 qualifiedComponentName.ComponentName + ".txt");
 
             List<string> text;
-            using (var codeModule = _projectsProvider.Component(qualifiedComponentName).CodeModule)
+            var component = _projectsProvider.Component(qualifiedComponentName);
+            using (var codeModule = component.CodeModule)
             {
                 text = codeModule.GetLines(1, codeModule.CountOfLines)
                     .Split(new[] {Environment.NewLine}, StringSplitOptions.None).ToList();
diff --git a/Rubberduck.Core/UI/Command/FindAllImplementationsCommand.cs b/Rubberduck.Core/UI/Command/FindAllImplementationsCommand.cs
@@ -188,7 +188,8 @@ private SearchResultsViewModel CreateViewModel(Declaration target)
 
         private string GetModuleLine(QualifiedModuleName module, int line)
         {
-            using (var codeModule = _state.ProjectsProvider.Component(module).CodeModule)
+            var component = _state.ProjectsProvider.Component(module);
+            using (var codeModule = component.CodeModule)
             {
                 return codeModule.GetLines(line, 1).Trim();
             }
diff --git a/Rubberduck.Core/UI/Command/FindAllReferencesCommand.cs b/Rubberduck.Core/UI/Command/FindAllReferencesCommand.cs
@@ -178,7 +178,8 @@ private SearchResultsViewModel CreateViewModel(Declaration declaration)
 
         private string GetModuleLine(QualifiedModuleName module, int line)
         {
-            using (var codeModule = _state.ProjectsProvider.Component(module).CodeModule)
+            var component = _state.ProjectsProvider.Component(module);
+            using (var codeModule = component.CodeModule)
             {
                 return codeModule.GetLines(line, 1).Trim();
             }
diff --git a/Rubberduck.Core/UI/Command/NavigateCommand.cs b/Rubberduck.Core/UI/Command/NavigateCommand.cs
@@ -31,7 +31,8 @@ protected override void OnExecute(object parameter)
 
             try
             {
-                using (var codeModule = _projectsProvider.Component(param.QualifiedName).CodeModule)
+                var component = _projectsProvider.Component(param.QualifiedName);
+                using (var codeModule = component.CodeModule)
                 {
                     using (var codePane = codeModule.CodePane)
                     {
diff --git a/Rubberduck.Core/UI/IdentifierReferences/IdentifierReferencesListDockablePresenter.cs b/Rubberduck.Core/UI/IdentifierReferences/IdentifierReferencesListDockablePresenter.cs
@@ -31,7 +31,8 @@ private void BindTarget(Declaration target)
 
         private void OnNavigateIdentifierReference(IdentifierReference reference)
         {
-            using (var codeModule = _projectsProvider.Component(reference.QualifiedModuleName).CodeModule)
+            var component = _projectsProvider.Component(reference.QualifiedModuleName);
+            using (var codeModule = component.CodeModule)
             {
                 using (var codePane = codeModule.CodePane)
                 {
diff --git a/Rubberduck.Core/UI/IdentifierReferences/ImplementationsListDockablePresenter.cs b/Rubberduck.Core/UI/IdentifierReferences/ImplementationsListDockablePresenter.cs
@@ -33,7 +33,8 @@ private void BindTarget(IEnumerable<Declaration> implementations)
 
         private void OnNavigateImplementation(Declaration implementation)
         {
-            using (var codeModule = _projectsProvider.Component(implementation.QualifiedName.QualifiedModuleName).CodeModule)
+            var component = _projectsProvider.Component(implementation.QualifiedName.QualifiedModuleName);
+            using (var codeModule = component.CodeModule)
             {
                 using (var codePane = codeModule.CodePane)
                 {
diff --git a/Rubberduck.Core/UI/ToDoItems/ToDoExplorerViewModel.cs b/Rubberduck.Core/UI/ToDoItems/ToDoExplorerViewModel.cs
@@ -150,7 +150,8 @@ public CommandBase RemoveCommand
                         return;
                     }
 
-                    using (var module = _state.ProjectsProvider.Component(_selectedItem.Selection.QualifiedName).CodeModule)
+                    var component = _state.ProjectsProvider.Component(_selectedItem.Selection.QualifiedName);
+                    using (var module = component.CodeModule)
                     {
                         var oldContent = module.GetLines(_selectedItem.Selection.Selection.StartLine, 1);
                         var newContent = oldContent.Remove(_selectedItem.Selection.Selection.StartColumn - 1);
diff --git a/Rubberduck.Core/UI/UnitTesting/Commands/AddTestMethodExpectedErrorCommand.cs b/Rubberduck.Core/UI/UnitTesting/Commands/AddTestMethodExpectedErrorCommand.cs
@@ -96,7 +96,8 @@ protected override void OnExecute(object parameter)
                 {
                     var declaration = _state.GetTestModules().FirstOrDefault(f =>
                     {
-                        using (var thisModule = _state.ProjectsProvider.Component(f.QualifiedName.QualifiedModuleName).CodeModule)
+                        var component = _state.ProjectsProvider.Component(f.QualifiedName.QualifiedModuleName);
+                        using (var thisModule = component.CodeModule)
                         {
                             return thisModule.Equals(activeModule);
                         }
diff --git a/Rubberduck.Parsing/Symbols/DeclarationSymbolsListener.cs b/Rubberduck.Parsing/Symbols/DeclarationSymbolsListener.cs
@@ -96,7 +96,8 @@ private IEnumerable<IAnnotation> FindAnnotations()
             }
 
             int lastDeclarationsSectionLine;
-            using (var codeModule = _state.ProjectsProvider.Component(_qualifiedModuleName).CodeModule)
+            var component = _state.ProjectsProvider.Component(_qualifiedModuleName);
+            using (var codeModule = component.CodeModule)
             {
                 lastDeclarationsSectionLine = codeModule.CountOfDeclarationLines;
             }
diff --git a/Rubberduck.Parsing/VBA/ParseErrorEventArgs.cs b/Rubberduck.Parsing/VBA/ParseErrorEventArgs.cs
@@ -19,7 +19,17 @@ public ParseErrorEventArgs(SyntaxErrorException exception, IVBComponent componen
 
         private readonly IVBComponent _component;
         public string ComponentName { get { return _component.Name; } }
-        public string ProjectName { get { return _component.Collection.Parent.Name; } }
+        public string ProjectName
+        {
+            get
+            {
+                using (var collection = _component.Collection)
+                using (var parent = collection.Parent)
+                {
+                    return parent.Name;
+                }
+            }
+        }
 
         public void Navigate()
         {
diff --git a/Rubberduck.Refactorings/ImplementInterface/ImplementInterfaceRefactoring.cs b/Rubberduck.Refactorings/ImplementInterface/ImplementInterfaceRefactoring.cs
@@ -81,7 +81,8 @@ public void Refactor(QualifiedSelection selection)
 
             if (oldSelection.HasValue)
             {
-                using (var module = _state.ProjectsProvider.Component(oldSelection.Value.QualifiedName).CodeModule)
+                var component = _state.ProjectsProvider.Component(oldSelection.Value.QualifiedName);
+                using (var module = component.CodeModule)
                 {
                     using (var pane = module.CodePane)
                     {
diff --git a/Rubberduck.Refactorings/IntroduceField/IntroduceFieldRefactoring.cs b/Rubberduck.Refactorings/IntroduceField/IntroduceFieldRefactoring.cs
@@ -86,7 +86,8 @@ private void PromoteVariable(IModuleRewriter rewriter, Declaration target)
 
             if (oldSelection.HasValue)
             {
-                using (var module = _state.ProjectsProvider.Component(oldSelection.Value.QualifiedName).CodeModule)
+                var component = _state.ProjectsProvider.Component(oldSelection.Value.QualifiedName);
+                using (var module = component.CodeModule)
                 {
                     using (var pane = module.CodePane)
                     {
diff --git a/Rubberduck.Refactorings/Rename/RenameRefactoring.cs b/Rubberduck.Refactorings/Rename/RenameRefactoring.cs
@@ -227,7 +227,8 @@ private bool IsValidTarget(Declaration target)
             }
             else if (target.DeclarationType.HasFlag(DeclarationType.Module))
             {
-                using (var module = _state.ProjectsProvider.Component(target.QualifiedName.QualifiedModuleName).CodeModule)
+                var component = _state.ProjectsProvider.Component(target.QualifiedName.QualifiedModuleName);
+                using (var module = component.CodeModule)
                 {
                     if (module.IsWrappingNullReference)
                     {
@@ -591,7 +592,8 @@ private IEnumerable<Declaration> FindEventHandlersForControl(Declaration control
 
         private void CacheInitialSelection(QualifiedSelection qSelection)
         {
-            using (var codeModule = _state.ProjectsProvider.Component(qSelection.QualifiedName).CodeModule)
+            var component = _state.ProjectsProvider.Component(qSelection.QualifiedName);
+            using (var codeModule = component.CodeModule)
             {
                 using (var codePane = codeModule.CodePane)
                 {
diff --git a/Rubberduck.Refactorings/ReorderParameters/ReorderParametersRefactoring.cs b/Rubberduck.Refactorings/ReorderParameters/ReorderParametersRefactoring.cs
@@ -153,7 +153,8 @@ private void AdjustReferences(IEnumerable<IdentifierReference> references)
                     continue; 
                 }
 
-                using (var module = _projectsProvider.Component(reference.QualifiedModuleName).CodeModule)
+                var component = _projectsProvider.Component(reference.QualifiedModuleName);
+                using (var module = component.CodeModule)
                 {
                     RewriteCall(argumentList, module);
                 }
diff --git a/RubberduckCodeAnalysis/ChainedWrapperAnalyzer.cs b/RubberduckCodeAnalysis/ChainedWrapperAnalyzer.cs
@@ -0,0 +1,57 @@
+using System.Collections.Immutable;
+using System.Linq;
+using Microsoft.CodeAnalysis;
+using Microsoft.CodeAnalysis.CSharp;
+using Microsoft.CodeAnalysis.CSharp.Syntax;
+using Microsoft.CodeAnalysis.Diagnostics;
+
+namespace RubberduckCodeAnalysis
+{
+    [DiagnosticAnalyzer(LanguageNames.CSharp)]
+    public class ChainedWrapperAnalyzer : DiagnosticAnalyzer
+    {
+        private const string ChainedWrapperId = "ChainedWrapper";
+        private static readonly DiagnosticDescriptor ChainedWrapperRule = new DiagnosticDescriptor(
+            ChainedWrapperId,
+            new LocalizableResourceString(nameof(Resources.ChainedWrapperTitle), Resources.ResourceManager, typeof(Resources)),
+            new LocalizableResourceString(nameof(Resources.ChainedWrapperMessageFormat), Resources.ResourceManager, typeof(Resources)),
+            new LocalizableResourceString(nameof(Resources.AnalyzerCategory), Resources.ResourceManager, typeof(Resources)).ToString(), 
+            DiagnosticSeverity.Error, 
+            true,
+            new LocalizableResourceString(nameof(Resources.ChainedWrapperDescription), Resources.ResourceManager, typeof(Resources))
+        );
+
+        public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics =>
+            ImmutableArray.Create(ChainedWrapperRule);
+
+        public override void Initialize(AnalysisContext context)
+        {
+            context.RegisterSyntaxNodeAction(AnalyzeSymbol, SyntaxKind.SimpleMemberAccessExpression);
+        }
+       
+        private static void AnalyzeSymbol(SyntaxNodeAnalysisContext context)
+        {
+            var node = (MemberAccessExpressionSyntax)context.Node;
+
+            if (!(node.Expression is InvocationExpressionSyntax || node.Expression is MemberAccessExpressionSyntax))
+            {
+                return;
+            }
+
+            var expInterfaces = context.SemanticModel.GetTypeInfo(node.Expression).Type?.AllInterfaces;
+            var nameInterfaces = context.SemanticModel.GetTypeInfo(node.Name).Type?.AllInterfaces;
+
+            if (!expInterfaces.HasValue || !nameInterfaces.HasValue)
+            {
+                return;
+            }
+
+            if (expInterfaces.Value.Any(a => a.ToDisplayString() == "Rubberduck.VBEditor.SafeComWrappers.Abstract.ISafeComWrapper") &&
+                nameInterfaces.Value.Any(a => a.ToDisplayString() == "Rubberduck.VBEditor.SafeComWrappers.Abstract.ISafeComWrapper"))
+            {
+                var diagnostic = Diagnostic.Create(ChainedWrapperRule, node.GetLocation());
+                context.ReportDiagnostic(diagnostic);
+            }
+        }
+    }
+}
diff --git a/RubberduckCodeAnalysis/Resources.Designer.cs b/RubberduckCodeAnalysis/Resources.Designer.cs
diff --git a/RubberduckCodeAnalysis/Resources.resx b/RubberduckCodeAnalysis/Resources.resx
@@ -120,6 +120,15 @@
   <data name="AnalyzerCategory" xml:space="preserve">
     <value>COM Management</value>
   </data>
+  <data name="ChainedWrapperDescription" xml:space="preserve">
+    <value>Chained Wrappers</value>
+  </data>
+  <data name="ChainedWrapperMessageFormat" xml:space="preserve">
+    <value>Chained Wrappers</value>
+  </data>
+  <data name="ChainedWrapperTitle" xml:space="preserve">
+    <value>Chained Wrappers</value>
+  </data>
   <data name="MissingClassInterfaceDescription" xml:space="preserve">
     <value>COM-visible classes must have an explicit ClassInterface attribute and be set to `None`. This is required to avoid versioning problems.</value>
   </data>
diff --git a/RubberduckCodeAnalysis/RubberduckCodeAnalysis.csproj b/RubberduckCodeAnalysis/RubberduckCodeAnalysis.csproj
@@ -144,9 +144,14 @@
     </Reference>
   </ItemGroup>
   <ItemGroup>
+    <Compile Include="ChainedWrapperAnalyzer.cs" />
     <Compile Include="ComVisibleTypeAnalyzer.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
-    <Compile Include="Resources.Designer.cs" />
+    <Compile Include="Resources.Designer.cs">
+      <AutoGen>True</AutoGen>
+      <DesignTime>True</DesignTime>
+      <DependentUpon>Resources.resx</DependentUpon>
+    </Compile>
     <Compile Include="RubberduckCodeAnalysisCodeFixProvider.cs" />
   </ItemGroup>
   <ItemGroup>
diff --git a/RubberduckTestsCodeAnalysis/RubberduckCodeAnalysisUnitTests.cs b/RubberduckTestsCodeAnalysis/RubberduckCodeAnalysisUnitTests.cs
diff --git a/RubberduckTestsCodeAnalysis/RubberduckTestsCodeAnalysis.csproj b/RubberduckTestsCodeAnalysis/RubberduckTestsCodeAnalysis.csproj

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,8 @@ protected override IEnumerable<IInspectionResult> DoGetInspectionResults()`
`30`	`30`	`foreach (var context in Listener.Contexts.Where(context => !IsIgnoringInspectionResultFor(context.ModuleName, context.Context.Start.Line)))`
`31`	`31`	`{`
`32`	`32`	`string lines;`
`33`		`- using (var module = State.ProjectsProvider.Component(context.ModuleName).CodeModule)`
	`33`	`+ var component = State.ProjectsProvider.Component(context.ModuleName);`
	`34`	`+ using (var module = component.CodeModule)`
`34`	`35`	`{`
`35`	`36`	`lines = module.GetLines(context.Context.Start.Line,`
`36`	`37`	`context.Context.Stop.Line - context.Context.Start.Line + 1);`
Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,8 @@ public CodeExplorerComponentViewModel(CodeExplorerItemViewModel parent, Declarat`
`110`	`110`
`111`	`111`	`private bool ContainsBuiltinDocumentPropertiesProperty()`
`112`	`112`	`{`
`113`		`- using (var properties = _projectsProvider.Component(Declaration.QualifiedName.QualifiedModuleName).Properties)`
	`113`	`+ var component = _projectsProvider.Component(Declaration.QualifiedName.QualifiedModuleName);`
	`114`	`+ using (var properties = component.Properties)`
`114`	`115`	`{`
`115`	`116`	`return properties.Any(item => item.Name == "BuiltinDocumentProperties");`
`116`	`117`	`}`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,11 @@ private IEnumerable<RegexSearchResult> GetResultsFromModule(ICodeModule module,`
`89`	`89`
`90`	`90`	`private void SetSelection(RegexSearchResult item)`
`91`	`91`	`{`
`92`		`- item.Module.CodePane.Selection = item.Selection;`
	`92`	`+ var module = item.Module;`
	`93`	`+ using (var codePane = module.CodePane)`
	`94`	`+ {`
	`95`	`+ codePane.Selection = item.Selection;`
	`96`	`+ }`
`93`	`97`	`}`
`94`	`98`
`95`	`99`	`private List<RegexSearchResult> SearchSelection(string searchPattern)`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,8 @@ protected override bool EvaluateCanExecute(object parameter)`
`33`	`33`
`34`	`34`	`try`
`35`	`35`	`{`
`36`		`- using (var codeModule = _projectsProvider.Component(node.Declaration.QualifiedName.QualifiedModuleName).CodeModule)`
	`36`	`+ var component = _projectsProvider.Component(node.Declaration.QualifiedName.QualifiedModuleName);`
	`37`	`+ using (var codeModule = component.CodeModule)`
`37`	`38`	`{`
`38`	`39`	`return codeModule.CountOfLines != 0;`
`39`	`40`	`}`
`@@ -54,7 +55,8 @@ protected override void OnExecute(object parameter)`
`54`	`55`	`qualifiedComponentName.ComponentName + ".txt");`
`55`	`56`
`56`	`57`	`List<string> text;`
`57`		`- using (var codeModule = _projectsProvider.Component(qualifiedComponentName).CodeModule)`
	`58`	`+ var component = _projectsProvider.Component(qualifiedComponentName);`
	`59`	`+ using (var codeModule = component.CodeModule)`
`58`	`60`	`{`
`59`	`61`	`text = codeModule.GetLines(1, codeModule.CountOfLines)`
`60`	`62`	`.Split(new[] {Environment.NewLine}, StringSplitOptions.None).ToList();`
Original file line number	Diff line number	Diff line change
`@@ -188,7 +188,8 @@ private SearchResultsViewModel CreateViewModel(Declaration target)`
`188`	`188`
`189`	`189`	`private string GetModuleLine(QualifiedModuleName module, int line)`
`190`	`190`	`{`
`191`		`- using (var codeModule = _state.ProjectsProvider.Component(module).CodeModule)`
	`191`	`+ var component = _state.ProjectsProvider.Component(module);`
	`192`	`+ using (var codeModule = component.CodeModule)`
`192`	`193`	`{`
`193`	`194`	`return codeModule.GetLines(line, 1).Trim();`
`194`	`195`	`}`
Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,8 @@ private SearchResultsViewModel CreateViewModel(Declaration declaration)`
`178`	`178`
`179`	`179`	`private string GetModuleLine(QualifiedModuleName module, int line)`
`180`	`180`	`{`
`181`		`- using (var codeModule = _state.ProjectsProvider.Component(module).CodeModule)`
	`181`	`+ var component = _state.ProjectsProvider.Component(module);`
	`182`	`+ using (var codeModule = component.CodeModule)`
`182`	`183`	`{`
`183`	`184`	`return codeModule.GetLines(line, 1).Trim();`
`184`	`185`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,8 @@ protected override void OnExecute(object parameter)`
`31`	`31`
`32`	`32`	`try`
`33`	`33`	`{`
`34`		`- using (var codeModule = _projectsProvider.Component(param.QualifiedName).CodeModule)`
	`34`	`+ var component = _projectsProvider.Component(param.QualifiedName);`
	`35`	`+ using (var codeModule = component.CodeModule)`
`35`	`36`	`{`
`36`	`37`	`using (var codePane = codeModule.CodePane)`
`37`	`38`	`{`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,8 @@ private void BindTarget(Declaration target)`
`31`	`31`
`32`	`32`	`private void OnNavigateIdentifierReference(IdentifierReference reference)`
`33`	`33`	`{`
`34`		`- using (var codeModule = _projectsProvider.Component(reference.QualifiedModuleName).CodeModule)`
	`34`	`+ var component = _projectsProvider.Component(reference.QualifiedModuleName);`
	`35`	`+ using (var codeModule = component.CodeModule)`
`35`	`36`	`{`
`36`	`37`	`using (var codePane = codeModule.CodePane)`
`37`	`38`	`{`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,8 @@ private void BindTarget(IEnumerable<Declaration> implementations)`
`33`	`33`
`34`	`34`	`private void OnNavigateImplementation(Declaration implementation)`
`35`	`35`	`{`
`36`		`- using (var codeModule = _projectsProvider.Component(implementation.QualifiedName.QualifiedModuleName).CodeModule)`
	`36`	`+ var component = _projectsProvider.Component(implementation.QualifiedName.QualifiedModuleName);`
	`37`	`+ using (var codeModule = component.CodeModule)`
`37`	`38`	`{`
`38`	`39`	`using (var codePane = codeModule.CodePane)`
`39`	`40`	`{`