kadircet updated this revision to Diff 134414.
kadircet added a comment.

Changed description of unit test to include direct link to bug.
Used different variable types to make sure alignment happens.


Repository:
  rC Clang

https://reviews.llvm.org/D43303

Files:
  unittests/Format/FormatTest.cpp


Index: unittests/Format/FormatTest.cpp
===================================================================
--- unittests/Format/FormatTest.cpp
+++ unittests/Format/FormatTest.cpp
@@ -9450,11 +9450,11 @@
       Alignment);
   Alignment.PointerAlignment = FormatStyle::PAS_Right;
 
-  // Bug 35641
+  // See llvm.org/PR35641
   Alignment.AlignConsecutiveDeclarations = true;
   verifyFormat("int func() { //\n"
-               "  int b;\n"
-               "  int c;\n"
+               "  int      b;\n"
+               "  unsigned c;\n"
                "}",
                Alignment);
 }


Index: unittests/Format/FormatTest.cpp
===================================================================
--- unittests/Format/FormatTest.cpp
+++ unittests/Format/FormatTest.cpp
@@ -9450,11 +9450,11 @@
       Alignment);
   Alignment.PointerAlignment = FormatStyle::PAS_Right;
 
-  // Bug 35641
+  // See llvm.org/PR35641
   Alignment.AlignConsecutiveDeclarations = true;
   verifyFormat("int func() { //\n"
-               "  int b;\n"
-               "  int c;\n"
+               "  int      b;\n"
+               "  unsigned c;\n"
                "}",
                Alignment);
 }
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to