[PATCH] D57739: [clangd] Format tweak's replacements.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 6 05:37:32 PST 2019
hokein added inline comments.
Comment at: clangd/refactor/Tweak.h:56
+ /// The style to format generated changes.
+ format::FormatStyle Style;
> NIT: Maybe make this a second argument of `apply`?
> This would convey the idea that `execute()` should not do formatting on its own.
SG, this also makes the `format` intention of `apply` more explicitly.
rCTE Clang Tools Extra
CHANGES SINCE LAST ACTION
More information about the cfe-commits