HazardyKnusperkeks accepted this revision.
HazardyKnusperkeks added a comment.
Remains the issue with the alignment, I would like to know @MyDeveloperDay 's
opinion on that. Should the values be right aligned, or left aligned? As far as
I see all alignment in clang-format is left until now.
================
Comment at: clang/lib/Format/WhitespaceManager.cpp:982
+void WhitespaceManager::alignArrayInitializers(unsigned Start, unsigned End) {
+
+ auto CellDescs = getCells(Start, End);
----------------
This is unneccessary. (But doesn't need a new revision until approved.)
================
Comment at: clang/lib/Format/WhitespaceManager.cpp:1081
+ unsigned End) {
+
+ unsigned Depth = 0;
----------------
Same
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101868/new/
https://reviews.llvm.org/D101868
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
