sammccall added a comment.

FWIW, I don't think this is worth adding an option or protocol extensions for 
at this point.

There are pros and cons to showing this in VSCode (pros are visibility without 
hovering, particularly in problems view and cross-editor consistency, cons are 
certainly noise for main-editor hover).
But this seems pretty minor either way.

If it turns out to be a big problem, a protocol extension and updating the 
editor extensions where relevant seems like a reasonable way to go, but it's 
certainly more complicated.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57509/new/

https://reviews.llvm.org/D57509



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

Reply via email to