[Lldb-commits] [PATCH] D40212: refactor: Unify+simplify DWARFCompileUnit ctor+Clear() into in-class initializers

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sun Nov 19 10:13:14 PST 2017


clayborg added a comment.

Good change in the header file. I am not sure I like the destruct this object in place and replace with new version... If this is commonly done and acceptable form of C++ I would be ok with it, but I agree with Pavel, it seems a little bit off the books.


https://reviews.llvm.org/D40212





More information about the lldb-commits mailing list