sammccall added a comment.

This borrows from https://reviews.llvm.org/D44226 (log levels) which got 
stalled for various reasons (@simark - is it OK if I pick this up?).
It doesn't actually split the LSP messages vs LSP message names into different 
levels, but that's a simple followup.

Switching to formatv seems like a nice ergonomic win while we're breaking 
interfaces.
It also in principle allows loggers to extract the format string and args 
separately (formatv doesn't yet provide APIs for this, but architecturally it's 
easy) which can help log analysis.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D49008



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

Reply via email to