Automatic code cleanup.

PiperOrigin-RevId: 415349792
bazelbuild · Dec 9, 2021 · ccc9b9d · ccc9b9d
1 parent 5c5f785
commit ccc9b9d
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 9 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/test/TestRunnerAction.java b/src/main/java/com/google/devtools/build/lib/analysis/test/TestRunnerAction.java
@@ -608,7 +608,7 @@ public boolean actionCacheHit(ActionCachedContext executor) {
                   .getContext(TestActionContext.class)
                   .newCachedTestResult(executor.getExecRoot(), this, cachedTestResultData.get()));
     } catch (IOException e) {
-      logger.atInfo().log(getErrorMessageOnNewCachedTestResultError(e.getMessage()));
+      logger.atInfo().log("%s", getErrorMessageOnNewCachedTestResultError(e.getMessage()));
       executor
           .getEventHandler()
           .handle(Event.warn(getErrorMessageOnNewCachedTestResultError(e.getMessage())));

diff --git a/src/main/java/com/google/devtools/build/lib/platform/SystemLoadAdvisoryModule.java b/src/main/java/com/google/devtools/build/lib/platform/SystemLoadAdvisoryModule.java
@@ -95,13 +95,13 @@ private synchronized void reportSystemLoadAdvisoryEvent(boolean isInitialValue,
 
     if (value < 0 || value > 100) {
       // values outside this range are not expected.
-      logger.atSevere().log(logString);
+      logger.atSevere().log("%s", logString);
     } else if (value > 50) {
       // 50 arbitrarily chosen as point where user is likely to be more concerned.
-      logger.atWarning().log(logString);
+      logger.atWarning().log("%s", logString);
     } else if (!isInitialValue || value > 25) {
       // Don't spam the logs if we have a Great or OK value at startup.
-      logger.atInfo().log(logString);
+      logger.atInfo().log("%s", logString);
     }
     if (reporter != null) {
       reporter.post(event);

diff --git a/src/main/java/com/google/devtools/build/lib/platform/SystemMemoryPressureMonitor.java b/src/main/java/com/google/devtools/build/lib/platform/SystemMemoryPressureMonitor.java
@@ -59,7 +59,7 @@ synchronized void memoryPressureCallback(int value) {
     if (reporter != null) {
       reporter.post(event);
     }
-    logger.atInfo().log(event.logString());
+    logger.atInfo().log("%s", event.logString());
     eventCount += 1;
   }
 }
diff --git a/src/main/java/com/google/devtools/build/lib/platform/SystemSuspensionModule.java b/src/main/java/com/google/devtools/build/lib/platform/SystemSuspensionModule.java
@@ -55,7 +55,7 @@ public synchronized void afterCommand() {
   /** Callback method called from JNI whenever a suspension event occurs. */
   synchronized void suspendCallback(int reason) {
     SystemSuspensionEvent event = new SystemSuspensionEvent(reason);
-    logger.atInfo().log(event.logString());
+    logger.atInfo().log("%s", event.logString());
     if (reporter != null) {
       reporter.post(event);
     }

diff --git a/src/main/java/com/google/devtools/build/lib/platform/SystemThermalModule.java b/src/main/java/com/google/devtools/build/lib/platform/SystemThermalModule.java
@@ -102,13 +102,13 @@ private synchronized void reportThermalEvent(boolean isInitialValue, int value)
 
     if (value < 0 || value > 100) {
       // values outside this range are not expected.
-      logger.atSevere().log(logString);
+      logger.atSevere().log("%s", logString);
     } else if (value > 50) {
       // 50 arbitrarily chosen as point where user is likely to be more concerned.
-      logger.atWarning().log(logString);
+      logger.atWarning().log("%s", logString);
     } else if (!isInitialValue || value != 0) {
       // Don't spam the logs if we have a nominal value at startup.
-      logger.atInfo().log(logString);
+      logger.atInfo().log("%s", logString);
     }
     if (reporter != null) {
       reporter.post(event);