Automatic code cleanup.

Googler · copybara-github · commit ccc9b9d65623 · 2021-12-09T13:58:55.000-08:00
PiperOrigin-RevId: 415349792
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);

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ synchronized void memoryPressureCallback(int value) {`
`59`	`59`	`if (reporter != null) {`
`60`	`60`	`reporter.post(event);`
`61`	`61`	`}`
`62`		`- logger.atInfo().log(event.logString());`
	`62`	`+ logger.atInfo().log("%s", event.logString());`
`63`	`63`	`eventCount += 1;`
`64`	`64`	`}`
`65`	`65`	`}`
Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public synchronized void afterCommand() {`
`55`	`55`	`/** Callback method called from JNI whenever a suspension event occurs. */`
`56`	`56`	`synchronized void suspendCallback(int reason) {`
`57`	`57`	`SystemSuspensionEvent event = new SystemSuspensionEvent(reason);`
`58`		`- logger.atInfo().log(event.logString());`
	`58`	`+ logger.atInfo().log("%s", event.logString());`
`59`	`59`	`if (reporter != null) {`
`60`	`60`	`reporter.post(event);`
`61`	`61`	`}`