clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.

One last thing: we should default construct a MemoryRegionInfo in all 
SBMemoryRegionInfo::SBMemoryRegionInfo() constructors so we don't have a make 
sure to call a non-const ref() method before it the pointer is valid. I know 
there is other code that does this, but any new code we have been updating to 
always default constructing an object into the std::unique_ptr just to be safe 
and avoid potential crashes.


http://reviews.llvm.org/D20565



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to