[Lldb-commits] [lldb] r128023 - in /lldb/trunk: include/lldb/Target/Process.h source/Host/common/Host.cpp source/Host/macosx/Host.mm

[Greg Clayton]

On Mar 21, 2011, at 2:45 PM, Stephen Wilson wrote:

> On Mon, Mar 21, 2011 at 09:25:07PM -0000, Greg Clayton wrote:
>> Author: gclayton
>> Date: Mon Mar 21 16:25:07 2011
>> New Revision: 128023
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=128023&view=rev
>> Log:
>> Added real user/group id, effective user/group id, and parent
>> process ID to the ProcessInfo.
>> 
>> 
>> Modified:
>>    lldb/trunk/include/lldb/Target/Process.h
>>    lldb/trunk/source/Host/common/Host.cpp
>>    lldb/trunk/source/Host/macosx/Host.mm
> 
> Hi Greg,
> 
> I was just looking at some of this Host stuff too. I need to add some
> more linux specific code (GetOSVersion) and it does not make sense to
> put that under common/.  I can start moving some of the other linux
> specific code out of the way too.  Any objections to creating a
> Host/linux subdirectory? 

No please do. I have abtracted more of the MacOSX specific Host calls over into Host.mm so it should clean up the common file a bit.