ilya-biryukov added inline comments.

================
Comment at: clangd/JSONRPCDispatcher.cpp:167
+      // It's another header, ignore it.
       continue;
+    } else {
----------------
puremourning wrote:
> ilya-biryukov wrote:
> > Maybe log the skipped header?
> I think this would just be noise for conforming clients which send the 
> (legitimate) Content-Type header. We could of course special case this, but 
> I'm not sure the extra logging would be all that useful.
You're right. Logging `Content-Type` headers doesn't make any sense.


https://reviews.llvm.org/D37282



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to