[PATCH] D73354: clang-format: insert trailing commas into containers.
Krasimir Georgiev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 24 07:55:21 PST 2020
krasimir added a comment.
General comment:
I think this is useful and could also be useful for other languages too (some folks have asked for this for some C++ code).
As this is a non-whitespace only change, it's a bit more risky than whitespace-only changes.
I think we need to establish a policy on such more riskier style options, at least have a convenient way for users to to opt-out of such and be extra careful including them by default in default styles.
This is tricky because some style guides explicitly recommend such changes.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73354/new/
https://reviews.llvm.org/D73354
More information about the cfe-commits
mailing list