[PATCH] D41031: [clangd] (Attempt to) read clang-format file for document formatting

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 12 00:49:46 PST 2017


sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

I'm not actually sure we're doing the user a service with this error handling.
LSP errors aren't surfaced to the user with a decent UI in practice.
Plus it adds a bunch of internal complexity.

I'd strongly consider just falling back to LLVM style on any error, inside ClangdServer::formatCode. I think the UX will be better, and the code will be simpler. But up to you.


https://reviews.llvm.org/D41031





More information about the cfe-commits mailing list