You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2022/01/01 14:27:11 UTC

[logging-log4j2] branch release-2.x updated: Constants are upper case.

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git


The following commit(s) were added to refs/heads/release-2.x by this push:
     new 2512cf2  Constants are upper case.
2512cf2 is described below

commit 2512cf20778b9d7f79927747266f74b3a3b9395b
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sat Jan 1 09:23:50 2022 -0500

    Constants are upper case.
---
 .../src/main/java/org/apache/log4j/builders/BuilderManager.java       | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/log4j-1.2-api/src/main/java/org/apache/log4j/builders/BuilderManager.java b/log4j-1.2-api/src/main/java/org/apache/log4j/builders/BuilderManager.java
index 12b0b3c..6e9b240 100644
--- a/log4j-1.2-api/src/main/java/org/apache/log4j/builders/BuilderManager.java
+++ b/log4j-1.2-api/src/main/java/org/apache/log4j/builders/BuilderManager.java
@@ -45,8 +45,8 @@ public class BuilderManager {
 
     public static final String CATEGORY = "Log4j Builder";
     private static final Logger LOGGER = StatusLogger.getLogger();
+    private static Class<?>[] CONSTRUCTOR_PARAMS = new Class[] { String.class, Properties.class };
     private final Map<String, PluginType<?>> plugins;
-    private static Class<?>[] constructorParams = new Class[] { String.class, Properties.class};
 
     public BuilderManager() {
         final PluginManager manager = new PluginManager(CATEGORY);
@@ -155,7 +155,7 @@ public class BuilderManager {
             if (AbstractBuilder.class.isAssignableFrom(clazz)) {
                 @SuppressWarnings("unchecked")
                 Constructor<T> constructor =
-                        (Constructor<T>) clazz.getConstructor(constructorParams);
+                        (Constructor<T>) clazz.getConstructor(CONSTRUCTOR_PARAMS);
                 return constructor.newInstance(prefix, props);
             }
             @SuppressWarnings("unchecked")