[Lldb-commits] [lldb] r129018 - in /lldb/trunk/source/Plugins/Platform/Linux: PlatformLinux.cpp PlatformLinux.h

Stephen Wilson wilsons at start.ca
Wed Apr 6 15:34:39 PDT 2011


On Wed, Apr 06, 2011 at 06:50:37PM -0000, Greg Clayton wrote:
> +PlatformLinux:: FindProcesses const ProcessInfoMatch &match_info,
> +                               ProcessInfoList &process_infos()
>  {
> -    return Host::FindProcessesByName (name_match, name_match_type, process_infos);
> +    return Host:: FindProcesses (match_info, process_infos);
>  }

Missing parens?

However, I think Marco's original patch was the right one (just delete
FindProcessesByName, now FindProcesses, from PlatformLinux) since the
base class handles the host case.  We only need to override when doing
remote debugging. 

No worries though, I can fix this up tonight :) 

-- 
steve




More information about the lldb-commits mailing list