[PATCH] D137181: [clang-format] Don't use 'PPIndentWidth' inside multi-line macros

Noah Goldstein via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 8 21:55:47 PST 2022


goldstein.w.n added a comment.

In D137181#3916474 <https://reviews.llvm.org/D137181#3916474>, @owenpan wrote:

> You can still simply the changes to UnwrappedLineParser.cpp a lot. In fact, instead of adjusting `PPLevel` at various places, you only need to set it once when `InMacroBody` is set.

I don't think thats right, it needs to stay updated with the true macro depth (even though we only use `PPLevel` for formatting `InMacroBody` lines).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137181



More information about the cfe-commits mailing list