How about calling it AbstractConfiguration now that it is abstract?

Gary

---------- Forwarded message ----------
From: <[email protected]>
Date: Sun, Mar 23, 2014 at 5:21 PM
Subject: svn commit: r1580614 -
/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
To: [email protected]


Author: mattsicker
Date: Sun Mar 23 21:21:41 2014
New Revision: 1580614

URL: http://svn.apache.org/r1580614
Log:
Make o.a.l.l.c.config.BaseConfiguration abstract.

  - Removed unused field.
  - Using this as an abstract class to help refactor these
  configuration classes.

Modified:

logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java

Modified:
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
URL:
http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java?rev=1580614&r1=1580613&r2=1580614&view=diff
==============================================================================
---
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
(original)
+++
logging/log4j/log4j2/trunk/log4j-core/src/main/java/org/apache/logging/log4j/core/config/BaseConfiguration.java
Sun Mar 23 21:21:41 2014
@@ -69,9 +69,7 @@ import org.apache.logging.log4j.util.Pro
 /**
  * The Base Configuration. Many configuration implementations will extend
this class.
  */
-public class BaseConfiguration extends AbstractFilterable implements
Configuration {
-
-    static final String PACKAGE_SEPARATOR = "\\s*,\\s*";
+public abstract class BaseConfiguration extends AbstractFilterable
implements Configuration {

     /**
      * Allow subclasses access to the status logger without creating
another instance.





-- 
E-Mail: [email protected] | [email protected]
Java Persistence with Hibernate, Second Edition<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Reply via email to