From de1423d662a57d5db49908da683b1e6eeb80f4a1 Mon Sep 17 00:00:00 2001 From: Ethan Atkins Date: Thu, 9 Jul 2020 09:14:52 -0700 Subject: [PATCH] Clarify boolean flag I found this difficult to read. --- .../main/scala/sbt/internal/nio/CheckBuildSources.scala | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/main/src/main/scala/sbt/internal/nio/CheckBuildSources.scala b/main/src/main/scala/sbt/internal/nio/CheckBuildSources.scala index 928f0ca21..326e08f15 100644 --- a/main/src/main/scala/sbt/internal/nio/CheckBuildSources.scala +++ b/main/src/main/scala/sbt/internal/nio/CheckBuildSources.scala @@ -92,12 +92,14 @@ private[sbt] class CheckBuildSources extends AutoCloseable { val filter = (c: String) => c == LoadProject || c == RebootCommand || c == TerminateAction || c == Shutdown || c.startsWith("sbtReboot") - val res = !commands.exists(filter) - if (!res) { + val resetState = commands.exists(filter) + if (resetState) { previousStamps.set(getStamps(force = true)) needUpdate.set(false) } - res + // We don't need to do a check since we just updated the stamps since + // we are about to perform a reload or reboot. + !resetState } @inline private def forceCheck = fileTreeRepository.isEmpty private[sbt] def needsReload(state: State, cmd: String) = {