First, we are talking about 6 lines of duplicated code. That isn't the kind of
"opportunity" I spend a lot of time thinking about :-)
I recall writing DefaultErrorHandler fairly quickly - I don't think much
thought went into it. Its primary goal is to try to reduce the number of
duplicate err
In org.apache.logging.log4j.core.appender.DefaultErrorHandler,
error(String, LogEvent, Throwable) looks the same as error(String,
Throwable)
Is that on purpose or the the LogEvent unused by mistake? Should it be
logged? Should it be saved in the exception?
With the code as is, there seems to be a