[all-commits] [llvm/llvm-project] c9cf39: [lldb] Replace NativeProcess delegate list with a ...

Pavel Labath via All-commits all-commits at lists.llvm.org
Tue Apr 13 00:50:13 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c9cf394f796e181b9ee3bc9b69d84fbbba2fe45c
      https://github.com/llvm/llvm-project/commit/c9cf394f796e181b9ee3bc9b69d84fbbba2fe45c
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2021-04-13 (Tue, 13 Apr 2021)

  Changed paths:
    M lldb/include/lldb/Host/common/NativeProcessProtocol.h
    M lldb/source/Host/common/NativeProcessProtocol.cpp
    M lldb/source/Plugins/Process/FreeBSD/NativeProcessFreeBSD.cpp
    M lldb/source/Plugins/Process/Linux/NativeProcessLinux.cpp
    M lldb/source/Plugins/Process/NetBSD/NativeProcessNetBSD.cpp

  Log Message:
  -----------
  [lldb] Replace NativeProcess delegate list with a single delegate

In all this time, we've never used more than one delegate. The logic to
support multiple delegates is therefore untested, and becomes
particularly unwieldy once we need to support multiple processes.

Just remove it.




More information about the All-commits mailing list