More explicitly clarify parenthesis logic

JustinGrote · Copilot · web-flow · commit b38a260b8bc0 · 2026-05-01T11:39:48.000-07:00
Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/src/PowerShellEditorServices/Services/TextDocument/RenameService.cs b/src/PowerShellEditorServices/Services/TextDocument/RenameService.cs
@@ -402,7 +402,7 @@ internal class RenameVariableVisitor(Ast target, string newName, bool skipVerify
     internal AstVisitAction Visit(Ast ast)
     {
         // If this is our first visit, we need to initialize and verify the scope, otherwise verify we are still on the same document.
-        if (!skipVerify && CurrentDocument is null || VariableDefinition is null)
+        if ((!skipVerify && CurrentDocument is null) || VariableDefinition is null)
         {
             CurrentDocument = ast.GetHighestParent();
             if (CurrentDocument.Find(ast => ast == target, true) is null)

Original file line number	Diff line number	Diff line change
`@@ -402,7 +402,7 @@ internal class RenameVariableVisitor(Ast target, string newName, bool skipVerify`
`402`	`402`	`internal AstVisitAction Visit(Ast ast)`
`403`	`403`	`{`
`404`	`404`	`// If this is our first visit, we need to initialize and verify the scope, otherwise verify we are still on the same document.`
`405`		`- if (!skipVerify && CurrentDocument is null \|\| VariableDefinition is null)`
	`405`	`+ if ((!skipVerify && CurrentDocument is null) \|\| VariableDefinition is null)`
`406`	`406`	`{`
`407`	`407`	`CurrentDocument = ast.GetHighestParent();`
`408`	`408`	`if (CurrentDocument.Find(ast => ast == target, true) is null)`