New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Method names should not be created via VariableRenamer #3231
Conversation
Converted to draft. As I am not sure it won't have side-effects when parsing interactively. I have an idea for a safer change. |
Hi, this PR fixes the issue for method names and should be safe. We should think about using a different separator for variable names (i.e. not '_' which is not allowed as Java Identifier, e.g. §. Otherwise we need to store leading zeros in the index names. But that is for a different PR. Best regards, |
78de5e0
to
f79fed5
Compare
Codecov Report
@@ Coverage Diff @@
## main #3231 +/- ##
============================================
+ Coverage 37.84% 37.85% +0.01%
- Complexity 16883 16892 +9
============================================
Files 2052 2052
Lines 125439 125443 +4
Branches 21215 21216 +1
============================================
+ Hits 47469 47487 +18
+ Misses 72117 72104 -13
+ Partials 5853 5852 -1
... and 4 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
ac5885c
to
7924752
Compare
This PR fixes #1717