[PATCH] D17910: clang-format: [JS] Handle certain cases of ASI.
Martin Probst via cfe-commits
cfe-commits at lists.llvm.org
Sun Mar 6 11:47:42 PST 2016
mprobst added inline comments.
Comment at: unittests/Format/FormatTestJS.cpp:56
@@ +55,3 @@
+ static void verifyFormatNoMessup(
+ llvm::StringRef Code,
> I don't think it is useful to do this as there are always ways in which formatting can fail leaving the input untouched. We still want to verify that formatting takes place.
> To do so, manually mess up the code in some way and use EXPECT_EQ. There should be various examples in FormatTest.cpp.
Done - introduced a `verifyFormat(str, str, style)` to make it a less verbose.
More information about the cfe-commits