[PATCH] D42173: [clangd] Simplify code handling compile commands

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 23 07:07:02 PST 2018


ilya-biryukov marked an inline comment as done.
ilya-biryukov added inline comments.


================
Comment at: clangd/ClangdUnit.h:62
+/// Information required to run clang (e.g., to parse AST or do code
+/// completion).
+struct ParseInputs {
----------------
sammccall wrote:
> sammccall wrote:
> > unwrap?
> This is not done. Is clang-format doing this?
> 
> (It looks like exactly 80 cols, but in any case the second comma shouldn't be there :-))
Sorry, my mistake. Fixed now.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D42173





More information about the cfe-commits mailing list