[Lldb-commits] [lldb] r230251 - Remove EOL whitespace from PlatformLinux

Ed Maste emaste at freebsd.org
Mon Feb 23 12:18:37 PST 2015


Author: emaste
Date: Mon Feb 23 14:18:37 2015
New Revision: 230251

URL: http://llvm.org/viewvc/llvm-project?rev=230251&view=rev
Log:
Remove EOL whitespace from PlatformLinux

This reduces the noise when diffing PlatformFreeBSD and PlatformLinux.

Modified:
    lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.cpp
    lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.h

Modified: lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.cpp?rev=230251&r1=230250&r2=230251&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.cpp (original)
+++ lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.cpp Mon Feb 23 14:18:37 2015
@@ -172,14 +172,14 @@ PlatformLinux::CreateInstance (bool forc
             default:
                 break;
         }
-        
+
         if (create)
         {
             switch (triple.getOS())
             {
                 case llvm::Triple::Linux:
                     break;
-                    
+
 #if defined(__linux__)
                 // Only accept "unknown" for the OS if the host is linux and
                 // it "unknown" wasn't specified (it was just returned because it
@@ -282,7 +282,7 @@ PlatformLinux::ResolveExecutable (const
 
     char exe_path[PATH_MAX];
     ModuleSpec resolved_module_spec (ms);
-    
+
     if (IsHost())
     {
         // If we have "ls" as the exe_file, resolve the executable location based on
@@ -314,7 +314,7 @@ PlatformLinux::ResolveExecutable (const
         else
         {
             // We may connect to a process and use the provided executable (Don't use local $PATH).
-            
+
             if (resolved_module_spec.GetFileSpec().Exists())
                 error.Clear();
             else
@@ -327,8 +327,8 @@ PlatformLinux::ResolveExecutable (const
         if (resolved_module_spec.GetArchitecture().IsValid())
         {
             error = ModuleList::GetSharedModule (resolved_module_spec,
-                                                 exe_module_sp, 
-                                                 NULL, 
+                                                 exe_module_sp,
+                                                 NULL,
                                                  NULL,
                                                  NULL);
             if (error.Fail())
@@ -348,14 +348,14 @@ PlatformLinux::ResolveExecutable (const
                         module_triple.setOSName (host_triple.getOSName());
 
                     error = ModuleList::GetSharedModule (resolved_module_spec,
-                                                         exe_module_sp, 
-                                                         NULL, 
+                                                         exe_module_sp,
+                                                         NULL,
                                                          NULL,
                                                          NULL);
                 }
             }
-        
-            // TODO find out why exe_module_sp might be NULL            
+
+            // TODO find out why exe_module_sp might be NULL
             if (!exe_module_sp || exe_module_sp->GetObjectFile() == NULL)
             {
                 exe_module_sp.reset();
@@ -373,11 +373,11 @@ PlatformLinux::ResolveExecutable (const
             for (uint32_t idx = 0; GetSupportedArchitectureAtIndex (idx, resolved_module_spec.GetArchitecture()); ++idx)
             {
                 error = ModuleList::GetSharedModule (resolved_module_spec,
-                                                     exe_module_sp, 
-                                                     NULL, 
+                                                     exe_module_sp,
+                                                     NULL,
                                                      NULL,
                                                      NULL);
-                // Did we find an executable using one of the 
+                // Did we find an executable using one of the
                 if (error.Success())
                 {
                     if (exe_module_sp && exe_module_sp->GetObjectFile())
@@ -385,12 +385,12 @@ PlatformLinux::ResolveExecutable (const
                     else
                         error.SetErrorToGenericError();
                 }
-                
+
                 if (idx > 0)
                     arch_names.PutCString (", ");
                 arch_names.PutCString (resolved_module_spec.GetArchitecture().GetArchitectureName());
             }
-            
+
             if (error.Fail() || !exe_module_sp)
             {
                 if (resolved_module_spec.GetFileSpec().Readable())
@@ -412,7 +412,7 @@ PlatformLinux::ResolveExecutable (const
 }
 
 Error
-PlatformLinux::GetFileWithUUID (const FileSpec &platform_file, 
+PlatformLinux::GetFileWithUUID (const FileSpec &platform_file,
                                 const UUID *uuid_ptr, FileSpec &local_file)
 {
     if (IsRemote())
@@ -455,7 +455,7 @@ PlatformLinux::GetProcessInfo (lldb::pid
     }
     else
     {
-        if (m_remote_platform_sp) 
+        if (m_remote_platform_sp)
             success = m_remote_platform_sp->GetProcessInfo (pid, process_info);
     }
     return success;
@@ -525,7 +525,7 @@ PlatformLinux::GetStatus (Stream &strm)
 }
 
 size_t
-PlatformLinux::GetSoftwareBreakpointTrapOpcode (Target &target, 
+PlatformLinux::GetSoftwareBreakpointTrapOpcode (Target &target,
                                                 BreakpointSite *bp_site)
 {
     ArchSpec arch = target.GetArchitecture();
@@ -537,7 +537,7 @@ PlatformLinux::GetSoftwareBreakpointTrap
     default:
         assert(false && "CPU type not supported!");
         break;
-            
+
     case llvm::Triple::aarch64:
         {
             static const uint8_t g_aarch64_opcode[] = { 0x00, 0x00, 0x20, 0xd4 };
@@ -573,8 +573,8 @@ PlatformLinux::GetSoftwareBreakpointTrap
             if (bp_loc_sp)
                 addr_class = bp_loc_sp->GetAddress ().GetAddressClass ();
 
-            if (addr_class == eAddressClassCodeAlternateISA 
-                || (addr_class == eAddressClassUnknown 
+            if (addr_class == eAddressClassCodeAlternateISA
+                || (addr_class == eAddressClassUnknown
                     && bp_loc_sp->GetAddress().GetOffset() & 1))
             {
                 trap_opcode = g_thumb_breakpoint_opcode;

Modified: lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.h
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.h?rev=230251&r1=230250&r2=230251&view=diff
==============================================================================
--- lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.h (original)
+++ lldb/trunk/source/Plugins/Platform/Linux/PlatformLinux.h Mon Feb 23 14:18:37 2015
@@ -30,7 +30,7 @@ namespace lldb_private {
 
         static void
         Terminate ();
-        
+
         PlatformLinux (bool is_host);
 
         virtual
@@ -50,7 +50,7 @@ namespace lldb_private {
 
         lldb_private::ConstString
         GetPluginName() override;
-        
+
         uint32_t
         GetPluginVersion() override
         {
@@ -89,7 +89,7 @@ namespace lldb_private {
         GetSupportedArchitectureAtIndex (uint32_t idx, ArchSpec &arch) override;
 
         size_t
-        GetSoftwareBreakpointTrapOpcode (Target &target, 
+        GetSoftwareBreakpointTrapOpcode (Target &target,
                                          BreakpointSite *bp_site) override;
 
         int32_t





More information about the lldb-commits mailing list