[lsp] Fix format string in Reply (#107480)

AmaranthineCodices · web-flow · commit e40c5b42fe8f · 2024-09-06T10:42:02.000+01:00
PR #105745 requires that `formatv` calls have the correct number of arguments. This call to `Logger::info` was incorrect.
diff --git 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<llvm::json::Value> 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));
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ void Reply::operator()(llvm::Expected<llvm::json::Value> reply) {`
`75`	`75`	`transport->reply(std::move(id), std::move(reply));`
`76`	`76`	`} else {`
`77`	`77`	`llvm::Error error = reply.takeError();`
`78`		`- Logger::info("--> reply:{0}({1})", method, id, error);`
	`78`	`+ Logger::info("--> reply:{0}({1}): {2}", method, id, error);`
`79`	`79`	`transport->reply(std::move(id), std::move(error));`
`80`	`80`	`}`
`81`	`81`	`}`