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