lldb::SBTarget and lldb::SBProcess are now thread hardened. They both still
contain shared pointers to the lldb_private::Target and lldb_private::Process
objects respectively as we won't want the target or process just going away.

Also cleaned up the lldb::SBModule to remove dangerous pointer accessors.

For any code the public API files, we should always be grabbing shared 
pointers to any objects for the current class, and any other classes prior
to running code with them.



git-svn-id: https://llvm.org/svn/llvm-project/lldb/trunk@149238 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/source/API/SBModule.cpp b/source/API/SBModule.cpp
index 28180fc..6200cc6 100644
--- a/source/API/SBModule.cpp
+++ b/source/API/SBModule.cpp
@@ -193,49 +193,18 @@
     return false;
 }
 
-lldb::ModuleSP &
-SBModule::operator *()
-{
-    return m_opaque_sp;
-}
-
-lldb_private::Module *
-SBModule::operator ->()
-{
-    return m_opaque_sp.get();
-}
-
-const lldb_private::Module *
-SBModule::operator ->() const
-{
-    return m_opaque_sp.get();
-}
-
-lldb_private::Module *
-SBModule::get()
-{
-    return m_opaque_sp.get();
-}
-
-const lldb_private::Module *
-SBModule::get() const
-{
-    return m_opaque_sp.get();
-}
-
-const lldb::ModuleSP &
-SBModule::get_sp() const
+ModuleSP
+SBModule::GetSP () const
 {
     return m_opaque_sp;
 }
 
 void
-SBModule::SetModule (const lldb::ModuleSP& module_sp)
+SBModule::SetSP (const ModuleSP &module_sp)
 {
     m_opaque_sp = module_sp;
 }
 
-
 SBAddress
 SBModule::ResolveFileAddress (lldb::addr_t vm_addr)
 {