Conversation
This reverts commit 32d9512.
| String org = variable; | ||
|
|
||
| String substValue = processSubstitution( | ||
| variable, org, cycleMap, configProps, callback, postprocessor, defaultsToEmptyString); |
There was a problem hiding this comment.
yes, should be covered by https://pmd.github.io/pmd/pmd_rules_java_bestpractices.html#unusedassignment
There was a problem hiding this comment.
seems its currently renaming not removal.
|
|
||
| private static String processSubstitution( | ||
| String variable, | ||
| final String variable, |
There was a problem hiding this comment.
boilerplate tradeoff naturally evolves to https://pmd.github.io/pmd/pmd_rules_java_bestpractices.html#avoidreassigningparameters.
There was a problem hiding this comment.
Agreed, I don't like final parameters.
There was a problem hiding this comment.
assuming you mean the explicit final declaration. Params should be treated final by default to avoid coupling, thus bugs.
|
Test failures might be related but unclear why it only fails on java 21 |
| private static String processSubstitution( | ||
| String variable, | ||
| String org, | ||
| final String variable, |
There was a problem hiding this comment.
| final String variable, | |
| String variable, |
gnodet
left a comment
gnodet
left a comment
There was a problem hiding this comment.
Remove the final parameter, otherwise looks good.
pr title? it seems more re |
|
No, it's definitely removed |
|
yes sorry from 7 to 6 params. |
3 participants
org is unclear and unneeded. Probably meant "original"? In any case, we don't need it.