[Lldb-commits] [PATCH] [lldb] Rename ThreadList to ProcessThreadList and abstract a superclass for a generic thread container

jingham at apple.com jingham at apple.com
Fri Sep 5 09:13:27 PDT 2014


In the case of breakpoint locations, I use "List" to mean a collection of things with other constraints - i.e. all belonging to the same breakpoint in the BreakpointLocation case, and "Collection" to mean a container of a bunch of stuff.  If it's not too much bother, it might be nice to keep using that convention?

Jim

> On Sep 4, 2014, at 7:08 PM, Kuba Brecka <kuba.brecka at gmail.com> wrote:
> 
> Based on the conversation at http://lists.cs.uiuc.edu/pipermail/lldb-commits/Week-of-Mon-20140901/012760.html this patch splits the current ThreadList class into a process-related thread object called "ProcessThreadList" and a generic thread list now called just ThreadList. The idea is to expose the ThreadList as SBThreadList and make ASan-related API based on it.
> 
> http://reviews.llvm.org/D5200
> 
> Files:
>  include/lldb/Target/OperatingSystem.h
>  include/lldb/Target/Process.h
>  include/lldb/Target/ProcessThreadList.h
>  include/lldb/Target/Thread.h
>  include/lldb/Target/ThreadList.h
>  include/lldb/lldb-forward.h
>  lldb.xcodeproj/project.pbxproj
>  source/Commands/CommandObjectTarget.cpp
>  source/Core/Debugger.cpp
>  source/Core/IOHandler.cpp
>  source/Plugins/ObjectFile/Mach-O/ObjectFileMachO.cpp
>  source/Plugins/OperatingSystem/Python/OperatingSystemPython.cpp
>  source/Plugins/OperatingSystem/Python/OperatingSystemPython.h
>  source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp
>  source/Plugins/Process/MacOSX-Kernel/ProcessKDP.h
>  source/Plugins/Process/elf-core/ProcessElfCore.cpp
>  source/Plugins/Process/elf-core/ProcessElfCore.h
>  source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
>  source/Plugins/Process/gdb-remote/ProcessGDBRemote.h
>  source/Plugins/Process/mach-core/ProcessMachCore.cpp
>  source/Plugins/Process/mach-core/ProcessMachCore.h
>  source/Target/Process.cpp
>  source/Target/ProcessThreadList.cpp
>  source/Target/Target.cpp
>  source/Target/ThreadList.cpp
> <D5200.13297.patch>_______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits




More information about the lldb-commits mailing list