Hey Gary,

wrong ML? ;-)

Regards,
Benedikt

Von meinem iPhone gesendet

Am 04.11.2012 um 15:06 schrieb Gary Gregory <garydgreg...@gmail.com>:

> I wonder if in:
> 
> org.apache.logging.log4j.core.Logger.log(Marker, String, Level, Message,
> Throwable)
> 
>    @Override
>    public void log(Marker marker, String fqcn, Level level, Message data,
> Throwable t) {
>        if (data == null) {
>            data = new SimpleMessage("");
>        }
>        config.config.getConfigurationMonitor().checkConfiguration();
>        config.loggerConfig.log(getName(), marker, fqcn, level, data, t);
>    }
> 
> The null guard should be pushed all the way down to:
> 
> org.apache.logging.log4j.core.impl.Log4jLogEvent.Log4jLogEvent(String,
> Marker, String, Level, Message, Throwable, Map<String, String>,
> ContextStack, String, StackTraceElement, long)
> 
> Which already does some constructor checks for other args.
> 
> ?
> 
> Gary
> -- 
> E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
> JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0
> Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK
> Blog: http://garygregory.wordpress.com
> Home: http://garygregory.com/
> Tweet! http://twitter.com/GaryGregory

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to