The rewriter contained two different operation visitor implementations depending on plus_operator configuration property.
If enabled (the default case) binary add, assign add and tpl expressions are rewritten among with call expressions.
If not enabled only call expressions are rewritten.
This PR removes the NoPlusOperatorVisitor
Motivation
The initial idea was to have two different visitor implementations for each case but the impossibility of one extending the other led to too much code duplication.
Additional Notes
Describe how to test your changes
Checklist
[ ] The CHANGELOG.md has been updated
[ ] Unit tests have been updated and pass
[ ] If known, an appropriate milestone has been selected
What does this PR do?
The rewriter contained two different operation visitor implementations depending on
plus_operatorconfiguration property.This PR removes the
NoPlusOperatorVisitorMotivation
The initial idea was to have two different visitor implementations for each case but the impossibility of one extending the other led to too much code duplication.
Additional Notes
Describe how to test your changes
Checklist