ksyx marked an inline comment as done. ksyx added inline comments.
================ Comment at: clang/unittests/Format/DefinitionBlockSeparatorTest.cpp:305 " int r = j / k;\n" + " if (struct S = getS()) {\n" + " // if condition\n" ---------------- curdeius wrote: > curdeius wrote: > > I'd rather see this added in another function e.g. bar3, because otherwise > > you don't test what was tested before. > When I wrote this comment I thought that bar3 didn't exist... But well, I > think I'll create a patch to clean up these tests a bit anyway. Thank you very much for this! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119067/new/ https://reviews.llvm.org/D119067 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits