[PATCH] D61438: [ASTImporter] Use llvm::Expected and Error in the importer API
Stefan Gränitz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 8 10:59:51 PDT 2019
sgraenitz added inline comments.
================
Comment at: lldb/source/Symbol/ClangASTImporter.cpp:68
+ return *ret_or_error;
+ } else {
+ Log *log =
----------------
aprantl wrote:
> The `else` is redundant.
Here it's necessary for the scope of `ret_or_error`. That's a bit unfortunate. Maybe invert the condition?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61438/new/
https://reviews.llvm.org/D61438
More information about the cfe-commits
mailing list