[PATCH] D33272: Method loadFromCommandLine should be able to report errors

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 22 06:31:21 PDT 2017


alexfh added inline comments.


================
Comment at: lib/Tooling/CompilationDatabase.cpp:164
+    // to user.
+    } else if (DiagLevel >= DiagnosticsEngine::Error)
+      Other.HandleDiagnostic(DiagLevel, Info);
----------------
Another couple of nits:
1. the comment below would probably be better moved to the body of the second `if`
2. enclose the next statement in braces.


https://reviews.llvm.org/D33272





More information about the cfe-commits mailing list