================ @@ -11450,6 +11450,11 @@ TEST_F(FormatTest, UnderstandsNewAndDelete) { "void new (link p);\n" "void delete (link p);"); + verifyFormat("{ p->delete(); }\n" + "{ p->new(); }", + "{ p->delete (); }\n" + "{ p->new (); }"); ---------------- scythris wrote:
Thank you for the comment @owenca Changing the order makes sense. But if the changes to `TokenAnnotator::spaceRequiredBetween` are reverted, the test without braces will not fail. That is why I included them. But it's not really clear to me why they are needed. https://github.com/llvm/llvm-project/pull/85470 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits