Code style fix

Author: George Hu

lldb/include/lldb/Core/ModuleSpec.h

@@ -274,9 +274,8 @@ class ModuleSpec {
   ArchSpec m_arch;
   UUID m_uuid;
   ConstString m_object_name;
-  std::weak_ptr<Target>
-      m_target; // This is set to take advantage of the target's search path
-                // and platform's locate module callback
+  /// This is set to take advantage of the target's search path and platform's locate module callback
+  std::weak_ptr<Target> m_target; 
   uint64_t m_object_offset = 0;
   uint64_t m_object_size = 0;
   llvm::sys::TimePoint<> m_object_mod_time;

lldb/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp

@@ -353,9 +353,8 @@ Status PlatformDarwin::GetSharedModule(
     // Get module search paths from the target if available.
     TargetSP target_sp = module_spec.GetTargetSP();
     FileSpecList module_search_paths;
-    if (target_sp) {
+    if (target_sp)
       module_search_paths = target_sp->GetExecutableSearchPaths();
-    }
     if (!module_sp && !module_search_paths.IsEmpty() && platform_file) {
       // We can try to pull off part of the file path up to the bundle
       // directory level and try any module search paths...
@@ -1310,9 +1309,8 @@ lldb_private::Status PlatformDarwin::FindBundleBinaryInExecSearchPaths(
   const FileSpec &platform_file = module_spec.GetFileSpec();
   TargetSP target_sp = module_spec.GetTargetSP();
   FileSpecList module_search_paths;
-  if (target_sp) {
+  if (target_sp)
     module_search_paths = target_sp->GetExecutableSearchPaths();
-  }
   // See if the file is present in any of the module_search_paths
   // directories.
   if (!module_sp && !module_search_paths.IsEmpty() && platform_file) {

lldb/source/Target/Target.cpp

@@ -1776,7 +1776,7 @@ bool Target::SetArchitecture(const ArchSpec &arch_spec, bool set_platform,
               arch_spec.GetArchitectureName(),
               arch_spec.GetTriple().getTriple().c_str());
     ModuleSpec module_spec(executable_sp->GetFileSpec(), other);
-    module_spec.SetTarget(this->shared_from_this());
+    module_spec.SetTarget(shared_from_this());
     Status error = ModuleList::GetSharedModule(module_spec, executable_sp,
                                                nullptr, nullptr);
 
@@ -2347,7 +2347,7 @@ ModuleSP Target::GetOrCreateModule(const ModuleSpec &orig_module_spec,
 
   // Apply any remappings specified in target.object-map:
   ModuleSpec module_spec(orig_module_spec);
-  module_spec.SetTarget(this->shared_from_this());
+  module_spec.SetTarget(shared_from_this());
   PathMappingList &obj_mapping = GetObjectPathMap();
   if (std::optional<FileSpec> remapped_obj_file =
           obj_mapping.RemapPath(orig_module_spec.GetFileSpec().GetPath(),
@@ -2406,7 +2406,7 @@ ModuleSP Target::GetOrCreateModule(const ModuleSpec &orig_module_spec,
           transformed_spec.GetFileSpec().SetDirectory(transformed_dir);
           transformed_spec.GetFileSpec().SetFilename(
                 module_spec.GetFileSpec().GetFilename());
-          transformed_spec.SetTarget(this->shared_from_this());
+          transformed_spec.SetTarget(shared_from_this());
           error = ModuleList::GetSharedModule(transformed_spec, module_sp,
                                               &old_modules, &did_create_module);
         }