mib accepted this revision. mib added a comment. This revision is now accepted and ready to land.
This patch looks good to me, however, may be it would be good to have a `IsInitialized`/ `IsCleared` method for all SB-types to make sure the object hasn't been cleared after going out-of-scope ? I'm suggesting this because not all SB-types have a `IsValid` method and for those who have it, the implementation varies a lot from class to class. So comparing the SB object against its default constructed value could be a way to check if the object was well-formed. Having the object reseted by SWIG if it goes out-of-scope could collide with this approach. What do you think ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116162/new/ https://reviews.llvm.org/D116162 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits