You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/11/30 15:53:35 UTC

[commons-codec] branch master updated: Lookup key in map only once

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


The following commit(s) were added to refs/heads/master by this push:
     new b323c01f Lookup key in map only once
b323c01f is described below

commit b323c01f0fd48ac9a1e8be3aaa3657cd3d9ab446
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Wed Nov 30 10:53:30 2022 -0500

    Lookup key in map only once
---
 .../org/apache/commons/codec/language/DaitchMokotoffSoundex.java     | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/commons/codec/language/DaitchMokotoffSoundex.java b/src/main/java/org/apache/commons/codec/language/DaitchMokotoffSoundex.java
index 1f502b38..b71a529d 100644
--- a/src/main/java/org/apache/commons/codec/language/DaitchMokotoffSoundex.java
+++ b/src/main/java/org/apache/commons/codec/language/DaitchMokotoffSoundex.java
@@ -361,8 +361,9 @@ public class DaitchMokotoffSoundex implements StringEncoder {
             }
 
             ch = Character.toLowerCase(ch);
-            if (folding && FOLDINGS.containsKey(ch)) {
-                ch = FOLDINGS.get(ch);
+            final Character character = FOLDINGS.get(ch);
+            if (folding && character != null) {
+                ch = character;
             }
             sb.append(ch);
         }