This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-codec.git
commit 0fdd68546238ae379da52f67a91afae58f4b7ef7 Author: Gary D. Gregory <[email protected]> AuthorDate: Sun Feb 9 18:00:53 2025 -0500 Add Checkstyle UnnecessaryParentheses --- src/conf/checkstyle.xml | 1 + src/main/java/org/apache/commons/codec/language/Metaphone.java | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/conf/checkstyle.xml b/src/conf/checkstyle.xml index d6322515..19b796c8 100644 --- a/src/conf/checkstyle.xml +++ b/src/conf/checkstyle.xml @@ -84,6 +84,7 @@ limitations under the License. <module name="SuperClone" /> <module name="SuperFinalize" /> <module name="TypecastParenPad" /> + <module name="UnnecessaryParentheses" /> <module name="UnusedImports" /> <module name="UpperEll" /> <module name="WhitespaceAfter" /> diff --git a/src/main/java/org/apache/commons/codec/language/Metaphone.java b/src/main/java/org/apache/commons/codec/language/Metaphone.java index ee3a0a30..34664e0d 100644 --- a/src/main/java/org/apache/commons/codec/language/Metaphone.java +++ b/src/main/java/org/apache/commons/codec/language/Metaphone.java @@ -262,7 +262,7 @@ public class Metaphone implements StringEncoder { code.append('K'); break; } - if (!isNextChar(local, n, 'H') || (n == 0 && wdsz >= 3 && isVowel(local, 2))) { // CH consonant -> K consonant + if (!isNextChar(local, n, 'H') || n == 0 && wdsz >= 3 && isVowel(local, 2)) { // CH consonant -> K consonant code.append('K'); } else { code.append('X'); // CHvowel -> X
