Doh!
Gary
On Nov 4, 2012, at 18:11, Benedikt Ritter <benerit...@gmail.com> wrote:
> 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
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
