[PATCH] D101868: [clang-format] Adds a formatter for aligning arrays of structs
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat May 8 03:31:51 PDT 2021
HazardyKnusperkeks added a comment.
Something which just came to my mind. Since you wrote your own LineFormatter, you have to add test cases for all kinds of indention and wrapping. It seems that the string wrapping does not happen, how about comments? Are classes, namespaces, functions, etc. correctly indented with various configurations?
Given my expectations I think the alignment has to happen differently, but I'm happy to be proven wrong.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101868/new/
https://reviews.llvm.org/D101868
More information about the cfe-commits
mailing list