The branch information being sent to the brancher.branch function in Executor for if statements and loop branching contains not quite the correct positional information. ce.condition is being used for the position information; however, ce.condition is potentially the negated version of the condition and so it is not directly an AST element that can be tracked. Removing the negation, even if the condition itself has negation should be okay in this case as either way an AST element for the correct location should be available if an outer negation is removed.
jennalwise
commented
2 years ago
The branch information being sent to the
brancher.branchfunction inExecutorfor if statements and loop branching contains not quite the correct positional information.ce.conditionis being used for the position information; however,ce.conditionis potentially the negated version of the condition and so it is not directly an AST element that can be tracked. Removing the negation, even if the condition itself has negation should be okay in this case as either way an AST element for the correct location should be available if an outer negation is removed.