[PATCH] D50154: [clangd] capitalize diagnostic messages if '-capitialize-diagnostic-text' is provided

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 2 04:52:17 PDT 2018


ioeric added a comment.

In https://reviews.llvm.org/D50154#1185545, @ilya-biryukov wrote:

> Maybe we could simply always capitalize the messages? Any cons to capitalizing error messages?
>  @simark, @malaperle, @ioeric, @hokein, WDYT?


Oh sorry, I thought `-capitialize-diagnostic-text` was an existing clang flag, in which case we should respect it. I don't think there is any con that I could think of, as long as it looks nice in clients :)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D50154





More information about the cfe-commits mailing list