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

Reply via email to