From e7018fe64238be5fc58f30740e096d6d2c2f905a Mon Sep 17 00:00:00 2001 From: Eugene Yokota Date: Mon, 21 Sep 2026 00:32:04 -0400 Subject: [PATCH] refactor: Avoid default parameter --- .../sbt/internal/server/NetworkChannel.scala | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/main/src/main/scala/sbt/internal/server/NetworkChannel.scala b/main/src/main/scala/sbt/internal/server/NetworkChannel.scala index 2cb8ff8c1..ff1773f2f 100644 --- a/main/src/main/scala/sbt/internal/server/NetworkChannel.scala +++ b/main/src/main/scala/sbt/internal/server/NetworkChannel.scala @@ -292,7 +292,7 @@ final class NetworkChannel( private[sbt] def respondError( err: JsonRpcResponseError, execId: Option[String], - skipExecRequests: Boolean = false + skipExecRequests: Boolean, ): Unit = this.synchronized { getPendingRequest(execId) match // a plain sbt/exec is answered when the command finishes, not from a task failure @@ -307,6 +307,13 @@ final class NetworkChannel( log.debug(msg) } + private[sbt] def respondError( + code: Long, + message: String, + execId: Option[String] + ): Unit = + respondError(JsonRpcResponseError(code, message), execId, skipExecRequests = false) + private[sbt] def getPendingRequest(execId: Option[String]): Option[JsonRpcRequestMessage] = execId.flatMap { // This handles multi commands from the network that were remapped to a different @@ -320,13 +327,6 @@ final class NetworkChannel( case id => pendingRequests.get(id) } - private[sbt] def respondError( - code: Long, - message: String, - execId: Option[String] - ): Unit = - respondError(JsonRpcResponseError(code, message), execId) - private[sbt] def respondResult[A: JsonFormat]( event: A, execId: Option[String]