logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Matt Sicker <boa...@gmail.com>
Subject Re: StatusConfiguration.initialize ()
Date Mon, 12 Sep 2016 18:25:56 GMT
Anything to reduce duplication of code is great! The Configuration classes
have quite a bit of duplication right now and are prime candidates for some
refactoring love.

On 28 August 2016 at 07:38, Mikael Ståldal <mikael.staldal@magine.com>
wrote:

> The method StatusConfiguration.initialize() is invoked from
> ConfigurationBuilder.build() and the constructors of XmlConfiguration,
> JsonConfiguration and CompositeConfiguration.
>
> Would it make sense if this was instead done in one place, in
> Configuration.initialize()? That would make the lifecycle of the
> configuration more consistent, first build it and then initialize it.
>



-- 
Matt Sicker <boards@gmail.com>

Mime
View raw message