jankratochvil marked 2 inline comments as done. jankratochvil added a comment.
Checked in, thanks for the review. ================ Comment at: lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:3987-4008 + DWARFASTParser *dwarf_ast = GetDWARFParser(*die.GetCU()); + if (dwarf_ast) + return dwarf_ast->GetDeclForUIDFromDWARF(die); + else + return CompilerDecl(); +} + ---------------- labath wrote: > [[ http://llvm.org/docs/CodingStandards.html#don-t-use-else-after-a-return | > Don't use else after return ]]. In fact, you might as well fold the > GetDWARFParser call into the if condition (`if (DWARFASTParser *parser = ...) > parser->...`) This was only moving the code, I did not expect to also refactor it. But I have thus made these code cleanups which I found. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D70646/new/ https://reviews.llvm.org/D70646 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
