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

Eric Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 31 10:18:30 PST 2019


ioeric added a comment.

In D57509#1378928 <https://reviews.llvm.org/D57509#1378928>, @ilya-biryukov wrote:

> Could this be made optional for VSCode?
>  As mentioned in the discussion before, I would personally prefer to turn it off, even though I do acknowledge the need for this for some clients, e.g. vim.


Yes, but a new option seems a bit of an overkill here.  The text is appended and doesn't seem to affect the readability of original diagnostic message much (IMO).  Could you elaborate why this is undesirable in vscode for you?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57509





More information about the llvm-commits mailing list