lawrence_danna added inline comments.
================ Comment at: lldb/bindings/python/python-typemaps.swig:500 + } +}; + ---------------- aadsm wrote: > labath wrote: > > Could you also `= delete` the copy operations to make sure nothing funny > > happens with those. > The `= delete` is unsupported in SWIG 2, only in 3: > http://www.swig.org/Doc3.0/CPlusPlus11.html#CPlusPlus11_defaulted_deleted > Do we really need it, or is there a workaround it, or should we just bump the > minimum requirements to SWIG 3? It shouldn't be strictly necessary. I put it in so if for some reason one of these values gets copied, it would result in a compiler error instead of a crash. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77480/new/ https://reviews.llvm.org/D77480 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
