PS: Don't double increment when looping through if clauses.

MathiasVP · MathiasVP · commit cc1348516fc6 · 2024-10-17T14:06:39.000+01:00
diff --git a/powershell/extractor/Semmle.Extraction.PowerShell/Entities/IfStatementEntity.cs b/powershell/extractor/Semmle.Extraction.PowerShell/Entities/IfStatementEntity.cs
@@ -21,7 +21,6 @@ public override void Populate(TextWriter trapFile)
                 var item1 = EntityConstructor.ConstructAppropriateEntity(PowerShellContext, Fragment.Clauses[index].Item1); 
                 var item2 = EntityConstructor.ConstructAppropriateEntity(PowerShellContext, Fragment.Clauses[index].Item2); 
                 trapFile.if_statement_clause(this, index, item1, item2);
-                index++; 
             }
             if (Fragment.ElseClause is not null)
             {

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,6 @@ public override void Populate(TextWriter trapFile)`
`21`	`21`	`var item1 = EntityConstructor.ConstructAppropriateEntity(PowerShellContext, Fragment.Clauses[index].Item1);`
`22`	`22`	`var item2 = EntityConstructor.ConstructAppropriateEntity(PowerShellContext, Fragment.Clauses[index].Item2);`
`23`	`23`	`trapFile.if_statement_clause(this, index, item1, item2);`
`24`		`- index++;`
`25`	`24`	`}`
`26`	`25`	`if (Fragment.ElseClause is not null)`
`27`	`26`	`{`