Skip to content

Commit 17bad86

Browse files
committed
Remove unnecessarily getting the module state in ResolveReferences
1 parent 1fa0edd commit 17bad86

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

Rubberduck.Parsing/VBA/RubberduckParser.cs

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -614,32 +614,34 @@ private Declaration CreateProjectDeclaration(QualifiedModuleName projectQualifie
614614
private void ResolveReferences(DeclarationFinder finder, VBComponent component, IParseTree tree)
615615
{
616616
Debug.Assert(State.Status == ParserState.ResolvedDeclarations);
617-
618-
var state = _state.GetModuleState(component);
617+
619618
var qualifiedName = new QualifiedModuleName(component);
620619
Logger.Debug("Resolving identifier references in '{0}'... (thread {1})", qualifiedName.Name, Thread.CurrentThread.ManagedThreadId);
620+
621621
var resolver = new IdentifierReferenceResolver(qualifiedName, finder);
622622
var listener = new IdentifierReferenceListener(resolver);
623+
623624
if (!string.IsNullOrWhiteSpace(tree.GetText().Trim()))
624625
{
625626
var walker = new ParseTreeWalker();
626627
try
627628
{
628-
Stopwatch watch = Stopwatch.StartNew();
629+
var watch = Stopwatch.StartNew();
629630
walker.Walk(listener, tree);
630631
watch.Stop();
631-
Logger.Debug("Binding Resolution done for component '{0}' in {1}ms (thread {2})", component.Name, watch.ElapsedMilliseconds, Thread.CurrentThread.ManagedThreadId);
632+
Logger.Debug("Binding Resolution done for component '{0}' in {1}ms (thread {2})", component.Name,
633+
watch.ElapsedMilliseconds, Thread.CurrentThread.ManagedThreadId);
634+
632635
_state.RebuildSelectionCache();
633-
state = ParserState.Ready;
636+
_state.SetModuleState(component, ParserState.Ready);
634637
}
635638
catch (Exception exception)
636639
{
637640
Logger.Error(exception, "Exception thrown resolving '{0}' (thread {1}).", component.Name, Thread.CurrentThread.ManagedThreadId);
638-
state = ParserState.ResolverError;
641+
_state.SetModuleState(component, ParserState.ResolverError);
639642
}
640643
}
641-
642-
_state.SetModuleState(component, state);
644+
643645
Logger.Debug("'{0}' is {1} (thread {2})", component.Name, _state.GetModuleState(component), Thread.CurrentThread.ManagedThreadId);
644646
}
645647

0 commit comments

Comments
 (0)