[PATCH] D28943: [clang-format] Remove redundant test in style-on-command-line.cpp

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 20 04:38:32 PST 2017


ioeric accepted this revision.
ioeric added a comment.
This revision is now accepted and ready to land.

@amaiorano: The test itself is correct. It's just that this test failed in our internal test. We could've fixed it internally, but the fix would be ugly. Since the intended behavior is already covered in the case above it, and it's really just checking the default fallback style is LLVM, which is not related to the original change, I think it makes sense to get rid of the case. Hope you don't mind :)


https://reviews.llvm.org/D28943





More information about the cfe-commits mailing list