[PATCH] D136154: [clang-format] Fix the continuation indenter

Henrik Lafrenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 19 00:17:27 PDT 2022


hel-ableton added a comment.

In D136154#3867328 <https://reviews.llvm.org/D136154#3867328>, @MyDeveloperDay wrote:

> Can we log a GitHub issue I can’t see what you are trying to fix

I'm sorry if this is unclear. The background to this is that our repository is currently formatted using clang-format 6.0.0, and we regard this as a bug that must have been introduced somewhere between 7.1.0 and 10.0.0, which is resulting in unnecessary reformatting of a lot of code, and is what kept us from upgrading clang-format for quite a while now. Other circumstances now force us to upgrade, though, and we would like to avoid introducing these changes, not only for diff noise, but also because we don't think the indentation makes sense that way.

A GitHub issue would be fine by me, also I could use some help with how to effectively work with the llvm-project repo, as currently I'm suffering a bit from huge turnaround times, like having to recompile lots and lots of code if I want to compare the result of formatting with or without the proposed fix.


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

https://reviews.llvm.org/D136154



More information about the cfe-commits mailing list