Skip to content

Commit ba8ee0b

Browse files
committed
Rename variables in ElementOrderCodeFixProvider
1 parent a32ff54 commit ba8ee0b

1 file changed

Lines changed: 12 additions & 12 deletions

File tree

StyleCop.Analyzers/StyleCop.Analyzers/OrderingRules/ElementOrderCodeFixProvider.cs

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -101,20 +101,20 @@ private static SyntaxNode UpdateSyntaxRoot(MemberDeclarationSyntax memberDeclara
101101

102102
private static SyntaxNode HandleTypeDeclaration(MemberOrderHelper memberOrder, TypeDeclarationSyntax typeDeclarationNode, ElementOrderingChecks checks, SyntaxNode syntaxRoot)
103103
{
104-
return MoveMember(memberOrder, typeDeclarationNode.Members, checks, syntaxRoot);
104+
return OrderMember(memberOrder, typeDeclarationNode.Members, checks, syntaxRoot);
105105
}
106106

107107
private static SyntaxNode HandleCompilationUnitDeclaration(MemberOrderHelper memberOrder, CompilationUnitSyntax compilationUnitDeclaration, ElementOrderingChecks checks, SyntaxNode syntaxRoot)
108108
{
109-
return MoveMember(memberOrder, compilationUnitDeclaration.Members, checks, syntaxRoot);
109+
return OrderMember(memberOrder, compilationUnitDeclaration.Members, checks, syntaxRoot);
110110
}
111111

112112
private static SyntaxNode HandleNamespaceDeclaration(MemberOrderHelper memberOrder, NamespaceDeclarationSyntax namespaceDeclaration, ElementOrderingChecks checks, SyntaxNode syntaxRoot)
113113
{
114-
return MoveMember(memberOrder, namespaceDeclaration.Members, checks, syntaxRoot);
114+
return OrderMember(memberOrder, namespaceDeclaration.Members, checks, syntaxRoot);
115115
}
116116

117-
private static SyntaxNode MoveMember(MemberOrderHelper memberOrder, SyntaxList<MemberDeclarationSyntax> members, ElementOrderingChecks checks, SyntaxNode syntaxRoot)
117+
private static SyntaxNode OrderMember(MemberOrderHelper memberOrder, SyntaxList<MemberDeclarationSyntax> members, ElementOrderingChecks checks, SyntaxNode syntaxRoot)
118118
{
119119
foreach (var member in members)
120120
{
@@ -130,18 +130,18 @@ private static SyntaxNode MoveMember(MemberOrderHelper memberOrder, SyntaxList<M
130130
return syntaxRoot;
131131
}
132132

133-
private static SyntaxNode MoveMember(SyntaxNode root, MemberDeclarationSyntax field, MemberDeclarationSyntax firstNonConst)
133+
private static SyntaxNode MoveMember(SyntaxNode root, MemberDeclarationSyntax member, MemberDeclarationSyntax targetMember)
134134
{
135-
var trackedRoot = root.TrackNodes(field, firstNonConst);
136-
var fieldToMove = trackedRoot.GetCurrentNode(field);
137-
var firstNonConstTracked = trackedRoot.GetCurrentNode(firstNonConst);
138-
if (!fieldToMove.HasLeadingTrivia)
135+
var trackedRoot = root.TrackNodes(member, targetMember);
136+
var memberToMove = trackedRoot.GetCurrentNode(member);
137+
var targetMemberTracked = trackedRoot.GetCurrentNode(targetMember);
138+
if (!memberToMove.HasLeadingTrivia)
139139
{
140-
fieldToMove = fieldToMove.WithLeadingTrivia(firstNonConstTracked.GetLeadingTrivia().Where(x => x.IsKind(SyntaxKind.WhitespaceTrivia)).LastOrDefault());
140+
memberToMove = memberToMove.WithLeadingTrivia(targetMemberTracked.GetLeadingTrivia().Where(x => x.IsKind(SyntaxKind.WhitespaceTrivia)).LastOrDefault());
141141
}
142142

143-
root = trackedRoot.InsertNodesBefore(firstNonConstTracked, new[] { fieldToMove });
144-
var fieldToMoveTracked = root.GetCurrentNodes(field).Last();
143+
root = trackedRoot.InsertNodesBefore(targetMemberTracked, new[] { memberToMove });
144+
var fieldToMoveTracked = root.GetCurrentNodes(member).Last();
145145
return root.RemoveNode(fieldToMoveTracked, SyntaxRemoveOptions.KeepNoTrivia);
146146
}
147147

0 commit comments

Comments
 (0)