From 3ad2d988d5e25022ad68b1cc9ff2dd3ef6a12043 Mon Sep 17 00:00:00 2001 From: Peter Weinberger Date: Wed, 19 Dec 2018 09:55:15 -0500 Subject: [PATCH] cmd/golsp: log elapsed times in milliseconds rather than microseconds. Language servers provided by Microsoft seem to use Date.Now() for calculating elapsed times. In javascript and typescript Date.Now() reports milliseconds. Change-Id: Ic12fc4a1d31f2548d8808b8b22aba80c44f000c3 Reviewed-on: https://go-review.googlesource.com/c/155018 Reviewed-by: Rebecca Stambler --- internal/lsp/cmd/server.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/internal/lsp/cmd/server.go b/internal/lsp/cmd/server.go index 81807abd..ed3b6e0b 100644 --- a/internal/lsp/cmd/server.go +++ b/internal/lsp/cmd/server.go @@ -97,7 +97,8 @@ func (s *server) Run(ctx context.Context, args ...string) error { } fmt.Fprint(outx, "'") if elapsed >= 0 { - fmt.Fprintf(outx, " in %vms", elapsed.Nanoseconds()/1000) + msec := int(elapsed.Round(time.Millisecond) / time.Millisecond) + fmt.Fprintf(outx, " in %dms", msec) } params := string(*payload) if params == "null" {