[PATCH] D28081: Make GetStyle return Expected<FormatStyle> instead of FormatStyle
Antonio Maiorano via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Dec 23 21:40:36 PST 2016
amaiorano added inline comments.
================
Comment at: unittests/Format/FormatTestObjC.cpp:72
TEST_F(FormatTestObjC, DetectsObjCInHeaders) {
- Style = getStyle("LLVM", "a.h", "none", "@interface\n"
+ Style = *getStyle("LLVM", "a.h", "none", "@interface\n"
"- (id)init;");
----------------
In these tests, I'm assuming getStyle returns a valid FormatSyle. I could add the same types of validation as in the FormatStyle.GetStyleOfFile tests.
https://reviews.llvm.org/D28081
More information about the cfe-commits
mailing list