[PATCH] D43303: [Format] Fix for bug 35641

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 15 04:32:50 PST 2018


djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.

Thanks for the fix.



================
Comment at: unittests/Format/FormatTest.cpp:9453
+
+  // Bug 35641
+  Alignment.AlignConsecutiveDeclarations = true;
----------------
Make this "See llvm.org/PR35641".


================
Comment at: unittests/Format/FormatTest.cpp:9457
+               "  int b;\n"
+               "  int c;\n"
+               "}",
----------------
Might be useful to use different types here to verify that alignment is actually happening, e.g. "int" and "unsigned".


Repository:
  rC Clang

https://reviews.llvm.org/D43303





More information about the cfe-commits mailing list