[all-commits] [llvm/llvm-project] 039cfe: [lldb/Target] Rename ThreadPlanPython into Scripte...

[Med Ismail Bennani via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 039cfe812c15c8f9e52ddb8cfc183fd8c927dba5
      https://github.com/llvm/llvm-project/commit/039cfe812c15c8f9e52ddb8cfc183fd8c927dba5
  Author: Med Ismail Bennani <ismail at bennani.ma>
  Date:   2024-08-05 (Mon, 05 Aug 2024)

  Changed paths:
    A lldb/include/lldb/Target/ScriptedThreadPlan.h
    R lldb/include/lldb/Target/ThreadPlanPython.h
    M lldb/source/API/SBThreadPlan.cpp
    M lldb/source/Target/CMakeLists.txt
    A lldb/source/Target/ScriptedThreadPlan.cpp
    M lldb/source/Target/Thread.cpp
    R lldb/source/Target/ThreadPlanPython.cpp

  Log Message:
  -----------
  [lldb/Target] Rename ThreadPlanPython into ScriptedThreadPlan (#101931)

Following 9a9ec228cdcf, since the ThreadPlanPython class started making
use of the Scripted Interface instead of calling directly into the
python methods, this class can work with other scripting languages (as
long as someone add the interfact for that language ;p).

So it doesn't make sense anymore for it to keep this name and also we
should avoid having language specific related classes outside the plugin
directory.

This patch renames the internal class from `ThreadPlanPython` to
`ScriptedThreadPlan` as its advertised externally, and also updates the
various log messages.

This should hopefully make the codebase more coherent.

Signed-off-by: Med Ismail Bennani <ismail at bennani.ma>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications