[PATCH] D57509: [clangd] Append "(fix available)" to diagnostic message when fixes are present.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 31 10:42:43 PST 2019


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





More information about the cfe-commits mailing list