From 8657be2d6b925168b6536075926f064a1d7f0db1 Mon Sep 17 00:00:00 2001 From: Lily Brown Date: Thu, 5 Sep 2024 15:39:27 -0700 Subject: [PATCH] [lsp] Fix format string in Reply PR #105745 requires that `formatv` calls have the correct number of arguments. This call to `Logger::info` was incorrect. --- mlir/lib/Tools/lsp-server-support/Transport.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mlir/lib/Tools/lsp-server-support/Transport.cpp b/mlir/lib/Tools/lsp-server-support/Transport.cpp index ca7ffdf78d3a1..ad8308f69aead 100644 --- a/mlir/lib/Tools/lsp-server-support/Transport.cpp +++ b/mlir/lib/Tools/lsp-server-support/Transport.cpp @@ -75,7 +75,7 @@ void Reply::operator()(llvm::Expected reply) { transport->reply(std::move(id), std::move(reply)); } else { llvm::Error error = reply.takeError(); - Logger::info("--> reply:{0}({1})", method, id, error); + Logger::info("--> reply:{0}({1}): {2}", method, id, error); transport->reply(std::move(id), std::move(error)); } }