slightly improved logging

Author: cmaglie

Diff for: handler/handler.go

@@ -129,18 +129,24 @@ func (handler *InoHandler) HandleMessageFromIDE(ctx context.Context, conn *jsonr
 		handler.synchronizer.DataMux.RUnlock()
 
 	case *lsp.DidOpenTextDocumentParams:
-		// method "textDocument/didOpen":
+		// method "textDocument/didOpen"
 		uri = p.TextDocument.URI
+		log.Printf("--> didOpen(%s)", uri)
+
 		handler.synchronizer.DataMux.Lock()
 		res, err := handler.didOpen(ctx, p)
 		handler.synchronizer.DataMux.Unlock()
+
 		if res == nil {
-			log.Println("    notification is not propagated to clangd")
+			log.Println("    --X notification is not propagated to clangd")
 			return nil, err // do not propagate to clangd
 		}
+
+		log.Printf("    --> didOpen(%s)", res.TextDocument.URI)
 		params = res
 
-	case *lsp.CompletionParams: // "textDocument/completion":
+	case *lsp.CompletionParams:
+		// method: "textDocument/completion"
 		uri = p.TextDocument.URI
 		log.Printf("--> completion(%s:%d:%d)\n", p.TextDocument.URI, p.Position.Line, p.Position.Character)
 
@@ -346,7 +352,6 @@ func (handler *InoHandler) didOpen(ctx context.Context, params *lsp.DidOpenTextD
 	// Add the TextDocumentItem in the tracked files list
 	doc := params.TextDocument
 	handler.trackedFiles[doc.URI] = doc
-	log.Printf("--> didOpen(%s)", doc.URI)
 
 	// If we are tracking a .ino...
 	if doc.URI.AsPath().Ext() == ".ino" {
@@ -364,7 +369,6 @@ func (handler *InoHandler) didOpen(ctx context.Context, params *lsp.DidOpenTextD
 					Version:    1,
 				},
 			}
-			log.Printf("    message for clangd: didOpen(%s)", newParam.TextDocument.URI)
 			return newParam, err
 		}
 	}

Diff for: handler/syncer.go

@@ -2,7 +2,6 @@ package handler
 
 import (
 	"context"
-	"log"
 	"sync"
 
 	"github.com/sourcegraph/jsonrpc2"
@@ -31,11 +30,11 @@ func (ah AsyncHandler) Handle(ctx context.Context, conn *jsonrpc2.Conn, req *jso
 			ah.synchronizer.FileMux.Lock()
 			defer ah.synchronizer.FileMux.Unlock()
 			if enableLogging {
-				log.Println("Message processing locked for", req.Method)
+				// log.Println("Message processing locked for", req.Method)
 			}
 			ah.handler.Handle(ctx, conn, req)
 			if enableLogging {
-				log.Println("Message processing unlocked for", req.Method)
+				// log.Println("Message processing unlocked for", req.Method)
 			}
 		}()
 	} else {