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-lang.git


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

commit 74ce18e283b7fcc05192aa7dc07f4c65741ad7d7
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Wed Nov 30 15:03:49 2022 -0500

    Lookup key in map only once
---
 src/main/java/org/apache/commons/lang3/ClassUtils.java | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/main/java/org/apache/commons/lang3/ClassUtils.java 
b/src/main/java/org/apache/commons/lang3/ClassUtils.java
index 8b15a3c37..1d2b9c461 100644
--- a/src/main/java/org/apache/commons/lang3/ClassUtils.java
+++ b/src/main/java/org/apache/commons/lang3/ClassUtils.java
@@ -528,13 +528,8 @@ public class ClassUtils {
      */
     public static Class<?> getClass(final ClassLoader classLoader, final 
String className, final boolean initialize) throws ClassNotFoundException {
         try {
-            final Class<?> clazz;
-            if (namePrimitiveMap.containsKey(className)) {
-                clazz = namePrimitiveMap.get(className);
-            } else {
-                clazz = Class.forName(toCanonicalName(className), initialize, 
classLoader);
-            }
-            return clazz;
+            Class<?> clazz = namePrimitiveMap.get(className);
+            return clazz != null ? clazz : 
Class.forName(toCanonicalName(className), initialize, classLoader);
         } catch (final ClassNotFoundException ex) {
             // allow path separators (.) as inner class name separators
             final int lastDotIndex = 
className.lastIndexOf(PACKAGE_SEPARATOR_CHAR);

Reply via email to