A common log4j mistake is to write something like:
try { // } catch( Exception ex ) { getLog().error( ex ); }
That, because the single arg method takes Object will write ex.toString() instead of a stacktrace.
So should we add one, i.e. in Log:
public void error( Exception ex ) { error( ex.getMessage(), ex ) }
Obviously adding null checking...