[PATCH] D57739: [clangd] Format tweak's replacements.
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 5 09:08:57 PST 2019
ilya-biryukov added a comment.
In D57739#1385144 <https://reviews.llvm.org/D57739#1385144>, @hokein wrote:
> This seems introduce intrusive changes to the Tweak interface (Tweak will need to know the `FormatStyle` somehow), I think this might be done in another patch.
It's totally fine, since we have just a single tweak now and changing the interface costs nothing.
The important part is making it **hard** to misuse the interface and forcing formatting on all the users of the interface is a significant burden. E.g. other users of `ClangdServer` (tests, users of clangd C++ API) shouldn't worry about formatting on their own.
Repository:
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57739/new/
https://reviews.llvm.org/D57739
More information about the cfe-commits
mailing list