Simplified logIfEnabled calls

This commit is contained in:
Collin Smith 2020-08-27 01:39:09 -07:00
parent e7f225f887
commit c4839f5e7e

View File

@ -35,6 +35,10 @@ abstract class AbstractLogger {
public abstract void level(final Level level);
public abstract boolean enabled(final Level level);
protected final void logIfEnabled(final Level level, final Message message) {
logIfEnabled(level, message, getLocation());
}
protected abstract void logIfEnabled(final Level level, final Message message, final StackTraceElement location);
public final boolean traceEnabled() {
@ -42,11 +46,11 @@ abstract class AbstractLogger {
}
public final void trace(final String message, final Object... params) {
logIfEnabled(Level.TRACE, defaultFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.TRACE, defaultFactory().newMessage(message, params));
}
public final void tracef(final String message, final Object... params) {
logIfEnabled(Level.TRACE, formattedFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.TRACE, formattedFactory().newMessage(message, params));
}
public final boolean debugEnabled() {
@ -54,11 +58,11 @@ abstract class AbstractLogger {
}
public final void debug(final String message, final Object... params) {
logIfEnabled(Level.DEBUG, defaultFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.DEBUG, defaultFactory().newMessage(message, params));
}
public final void debugf(final String message, final Object... params) {
logIfEnabled(Level.DEBUG, formattedFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.DEBUG, formattedFactory().newMessage(message, params));
}
public final boolean infoEnabled() {
@ -66,11 +70,11 @@ abstract class AbstractLogger {
}
public final void info(final String message, final Object... params) {
logIfEnabled(Level.INFO, defaultFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.INFO, defaultFactory().newMessage(message, params));
}
public final void infof(final String message, final Object... params) {
logIfEnabled(Level.INFO, formattedFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.INFO, formattedFactory().newMessage(message, params));
}
public final boolean warnEnabled() {
@ -78,11 +82,11 @@ abstract class AbstractLogger {
}
public final void warn(final String message, final Object... params) {
logIfEnabled(Level.WARN, defaultFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.WARN, defaultFactory().newMessage(message, params));
}
public final void warnf(final String message, final Object... params) {
logIfEnabled(Level.WARN, formattedFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.WARN, formattedFactory().newMessage(message, params));
}
public final boolean errorEnabled() {
@ -90,11 +94,11 @@ abstract class AbstractLogger {
}
public final void error(final String message, final Object... params) {
logIfEnabled(Level.ERROR, defaultFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.ERROR, defaultFactory().newMessage(message, params));
}
public final void errorf(final String message, final Object... params) {
logIfEnabled(Level.ERROR, formattedFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.ERROR, formattedFactory().newMessage(message, params));
}
public final boolean fatalEnabled() {
@ -102,11 +106,11 @@ abstract class AbstractLogger {
}
public final void fatal(final String message, final Object... params) {
logIfEnabled(Level.FATAL, defaultFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.FATAL, defaultFactory().newMessage(message, params));
}
public final void fatalf(final String message, final Object... params) {
logIfEnabled(Level.FATAL, formattedFactory().newMessage(message, params), getLocation());
logIfEnabled(Level.FATAL, formattedFactory().newMessage(message, params));
}
private StackTraceElement getLocation() {