I like it. Will include in the refactoring I'm working on.

On 23 March 2014 19:12, Gary Gregory <[email protected]> wrote:

> 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
>



-- 
Matt Sicker <[email protected]>

Reply via email to