commit b3f72b3ec208af05ad51448aad8a0d67367d7f16
author David Pursehouse <dpursehouse@collab.net> Wed Feb 19 11:28:26 2020 +0900
committer David Pursehouse <dpursehouse@collab.net> Wed Feb 19 11:28:26 2020 +0900
tree d8f64414f3eec5c54c981a07e19ca4a45f8f1ad4
parent 3d4e60ec41e0d66b6e843cbad531e5a91de0138d

BlameServlet: Use logger's built-in formatting rather than String.format

Change-Id: I2694145030a4b5734563d8ca3de6df91bc865a15

java/com/google/gitiles/blame/BlameServlet.java

diff --git a/java/com/google/gitiles/blame/BlameServlet.java b/java/com/google/gitiles/blame/BlameServlet.java
index 8614c6a..ab9cd95 100644
--- a/java/com/google/gitiles/blame/BlameServlet.java
+++ b/java/com/google/gitiles/blame/BlameServlet.java
@@ -165,14 +165,13 @@
 
     if (!Objects.equals(currCommitBlobId, lastCommitBlobId)) {
       log.warn(
-          String.format(
-              "Blob %s in last modified commit %s for repo %s starting from %s"
-                  + " does not match original blob %s",
-              ObjectId.toString(lastCommitBlobId),
-              ObjectId.toString(lastCommit),
-              access.getRepositoryName(),
-              ObjectId.toString(currCommit),
-              ObjectId.toString(currCommitBlobId)));
+          "Blob {} in last modified commit {} for repo {} starting from {}"
+              + " does not match original blob {}",
+          ObjectId.toString(lastCommitBlobId),
+          ObjectId.toString(lastCommit),
+          access.getRepositoryName(),
+          ObjectId.toString(currCommit),
+          ObjectId.toString(currCommitBlobId));
       lastCommitBlobId = currCommitBlobId;
       lastCommit = currCommit;
     }