[PATCH] D53072: [clang-format] Create a new tool for IDEs based on clang-format

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 25 05:45:17 PDT 2019


sammccall added a comment.

You may be interested in D60605 <https://reviews.llvm.org/D60605>, which is a related idea (adding incremental format-and-indent-on-type to clangd).

These are opposite extremes in some sense: this patch integrates deeply into clang-format and that patch entirely layers on top of it, by transforming the source code (in hacky ways, in some cases).

I think the cleanest approach is probably some combination: in particular a "preserve line break at point" primitive does seem useful, and even if source code transforms are used, putting a nice API on this in clang-format would be nice. (I'm not sure a separate command-line-tool is justified, though).

If you haven't run into these yet, here are some fun issues I ran into:

- clang-format bails out when brackets aren't sufficiently matched, incremental formatting needs to work in such cases
- incremental formatting sometimes wants to make edits both immediately before and after the cursor. `tooling::Replacements` can't represent this, they will be merged and destroy precise cursor position information.
- user expectations when breaking between `()` are fairly clear. `{}` is the same when it acts as a list, but not when it acts as a block!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D53072/new/

https://reviews.llvm.org/D53072





More information about the cfe-commits mailing list