logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ralph Goers <ralph.go...@dslextreme.com>
Subject Re: [v2] org.apache.logging.log4j.core.Logger.log(Marker, String, Level, Message, Throwable)
Date Mon, 05 Nov 2012 01:15:42 GMT
I suppose it might be safer although it appears that the only code path to create an event
(other than in test classes) is through this code.

Ralph

On Nov 4, 2012, at 4:56 PM, Gary Gregory wrote:

> Hi All:
> 
> 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: garydgregory@gmail.com | ggregory@apache.org 
> JUnit in Action, 2nd Ed: http://bit.ly/ECvg0
> Spring Batch in Action: http://bit.ly/bqpbCK
> Blog: http://garygregory.wordpress.com 
> Home: http://garygregory.com/
> Tweet! http://twitter.com/GaryGregory
> 


Mime
View raw message