Fix FloggerArgumentToString bug pattern flagged by error prone Details: https://errorprone.info/bugpattern/FloggerArgumentToString Change-Id: I20e756c4de965964c1d6a301a20913c6d2a4153e
diff --git a/src/main/java/com/googlesource/gerrit/plugins/webhooks/PostTask.java b/src/main/java/com/googlesource/gerrit/plugins/webhooks/PostTask.java index 354bdf8..c069e3a 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/webhooks/PostTask.java +++ b/src/main/java/com/googlesource/gerrit/plugins/webhooks/PostTask.java
@@ -88,7 +88,7 @@ logRetry(e); reschedule(); } else { - log.atSevere().withCause(e).log("Failed to post: %s", toString()); + log.atSevere().withCause(e).log("Failed to post: %s", this); } } } @@ -98,12 +98,11 @@ } private void logRetry(String reason) { - log.atFine().log( - "Retrying %s in %dms. Reason: %s", toString(), remote.getRetryInterval(), reason); + log.atFine().log("Retrying %s in %dms. Reason: %s", this, remote.getRetryInterval(), reason); } private void logRetry(Throwable cause) { - log.atFine().withCause(cause).log("Retrying %s in %dms", toString(), remote.getRetryInterval()); + log.atFine().withCause(cause).log("Retrying %s in %dms", this, remote.getRetryInterval()); } @Override