csmulhern marked an inline comment as done.
csmulhern added inline comments.
================
Comment at: clang/unittests/Format/FormatTest.cpp:22414
+
+ EXPECT_EQ("int a = (int)b;", format("int a = (\n"
+ " int\n"
----------------
MyDeveloperDay wrote:
> can't this just be verifyFormat but with 3 arguments?
Isn't that true of all the EXPECT_EQ cases (as long as the expected code is
stable)? This is my first contribution and I'm basically cargo cutting what I
see as the prevailing style (verifyFormat for the one param / two param case,
EXPECT_EQ for the three param case). I can imagine that verifyFormat didn't
support the three parameter case before and so the EXPECT_EQ usage is legacy
code. Should I convert everything to use verifyFormat?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109557/new/
https://reviews.llvm.org/D109557
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
