[Lldb-commits] [PATCH] D45700: Improve LLDB's handling of non-local minidumps

Adrian McCarthy via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Apr 18 13:51:31 PDT 2018

amccarth accepted this revision.
amccarth added a comment.

I actually preferred the factory solution.

What I intended to express is that the modification of the target doesn't seems like it should be inside the PlaceholderModule class, so whether you use a factory or not doesn't really address that aspect of the coupling.  In my head, modification of the target should be done by the client that instantiates the PlaceholderModule (whether it does that via constructor or factory).

But this isn't a new problem, so I'm happy to consider the coupling problem outside the scope of this patch.


More information about the lldb-commits mailing list