tammela added inline comments.
================ Comment at: lldb/bindings/lua/lua-wrapper.swig:25-29 + auto extra_args = [&]() -> llvm::Optional<lldb::SBStructuredData> { + if (extra_args_impl == nullptr) + return {}; + return lldb::SBStructuredData(extra_args_impl); + } (); ---------------- JDevlieghere wrote: > I don't think you need the lambda. I used `unique_ptr` but `llvm:Optional` > would work as well. `{}` is not valid on the ternary operator, but I think the version suits your suggestion. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93649/new/ https://reviews.llvm.org/D93649 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits