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

Henrik Lafrenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 19 05:19:09 PDT 2022


hel-ableton added a comment.

> The problem as I see it was that the original bug, highly constrained the cases where "CurrentState.LastSpace = State.Column;" to one particular style (which if it happens to be your style great but not if its not.

You mean the original bugfix was already unnecessarily constrained, and now my proposed fix is only opening it up for one my case? That may be so. In any case this might really not be the ideal fix, and I'm open to any other, better way of fixing it.


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

https://reviews.llvm.org/D136154



More information about the cfe-commits mailing list