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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits