Skip to content

Commit 06d9d72

Browse files
committed
FindAllReferencesCommand:
- Reduced indentation levels
1 parent 7d62db8 commit 06d9d72

File tree

1 file changed

+21
-23
lines changed

1 file changed

+21
-23
lines changed

RetailCoder.VBE/UI/Command/FindAllReferencesCommand.cs

Lines changed: 21 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -43,11 +43,11 @@ public FindAllReferencesCommand(INavigateCommand navigateCommand, IMessageBox me
4343
private Declaration FindNewDeclaration(Declaration declaration)
4444
{
4545
return _state.DeclarationFinder
46-
.MatchName(declaration.IdentifierName)
47-
.SingleOrDefault(d => d.ProjectId == declaration.ProjectId
48-
&& d.ComponentName == declaration.ComponentName
49-
&& d.ParentScope == declaration.ParentScope
50-
&& d.DeclarationType == declaration.DeclarationType);
46+
.MatchName(declaration.IdentifierName)
47+
.SingleOrDefault(d => d.ProjectId == declaration.ProjectId
48+
&& d.ComponentName == declaration.ComponentName
49+
&& d.ParentScope == declaration.ParentScope
50+
&& d.DeclarationType == declaration.DeclarationType);
5151
}
5252

5353
private void _state_StateChanged(object sender, ParserStateEventArgs e)
@@ -162,8 +162,8 @@ private Declaration FindTarget(object parameter)
162162
}
163163

164164
return _vbe.ActiveCodePane != null && (_vbe.SelectedVBComponent?.HasDesigner ?? false)
165-
? FindFormDesignerTarget()
166-
: FindCodePaneTarget();
165+
? FindFormDesignerTarget()
166+
: FindCodePaneTarget();
167167
}
168168

169169
private Declaration FindCodePaneTarget()
@@ -173,36 +173,34 @@ private Declaration FindCodePaneTarget()
173173

174174
private Declaration FindFormDesignerTarget(QualifiedModuleName? qualifiedModuleName = null)
175175
{
176-
(var projectId, var component)
177-
= qualifiedModuleName.HasValue
178-
? (qualifiedModuleName.Value.ProjectId, qualifiedModuleName.Value.Component)
179-
: (_vbe.ActiveVBProject.ProjectId, _vbe.SelectedVBComponent);
176+
(var projectId, var component) = qualifiedModuleName.HasValue
177+
? (qualifiedModuleName.Value.ProjectId, qualifiedModuleName.Value.Component)
178+
: (_vbe.ActiveVBProject.ProjectId, _vbe.SelectedVBComponent);
180179

181180
if (component?.HasDesigner ?? false)
182181
{
183182
if (qualifiedModuleName.HasValue)
184183
{
185184
return _state.DeclarationFinder
186-
.MatchName(qualifiedModuleName.Value.Name)
187-
.SingleOrDefault(m => m.ProjectId == projectId
188-
&& m.DeclarationType.HasFlag(qualifiedModuleName.Value.ComponentType)
189-
&& m.ComponentName == component.Name);
185+
.MatchName(qualifiedModuleName.Value.Name)
186+
.SingleOrDefault(m => m.ProjectId == projectId
187+
&& m.DeclarationType.HasFlag(qualifiedModuleName.Value.ComponentType)
188+
&& m.ComponentName == component.Name);
190189
}
191190

192191
var selectedCount = component.SelectedControls.Count;
193192
if (selectedCount > 1) { return null; }
194193

195194
// Cannot use DeclarationType.UserForm, parser only assigns UserForms the ClassModule flag
196-
(var selectedType, var selectedName)
197-
= selectedCount == 0
198-
? (DeclarationType.ClassModule, component.Name)
199-
: (DeclarationType.Control, component.SelectedControls[0].Name);
195+
(var selectedType, var selectedName) = selectedCount == 0
196+
? (DeclarationType.ClassModule, component.Name)
197+
: (DeclarationType.Control, component.SelectedControls[0].Name);
200198

201199
return _state.DeclarationFinder
202-
.MatchName(selectedName)
203-
.SingleOrDefault(m => m.ProjectId == projectId
204-
&& m.DeclarationType.HasFlag(selectedType)
205-
&& m.ComponentName == component.Name);
200+
.MatchName(selectedName)
201+
.SingleOrDefault(m => m.ProjectId == projectId
202+
&& m.DeclarationType.HasFlag(selectedType)
203+
&& m.ComponentName == component.Name);
206204
}
207205
return null;
208206
}

0 commit comments

Comments
 (0)