[all-commits] [llvm/llvm-project] d0e37d: [lldb-dap] Refactor request handlers (NFC) (#128262)

[Jonas Devlieghere via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d0e37d9723314199e08e3bb8a61f9b7e033b1ff4
      https://github.com/llvm/llvm-project/commit/d0e37d9723314199e08e3bb8a61f9b7e033b1ff4
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2025-02-23 (Sun, 23 Feb 2025)

  Changed paths:
    M lldb/tools/lldb-dap/CMakeLists.txt
    M lldb/tools/lldb-dap/DAP.cpp
    M lldb/tools/lldb-dap/DAP.h
    A lldb/tools/lldb-dap/EventHelper.cpp
    A lldb/tools/lldb-dap/EventHelper.h
    A lldb/tools/lldb-dap/Handler/AttachRequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/BreakpointLocationsHandler.cpp
    A lldb/tools/lldb-dap/Handler/CompletionsHandler.cpp
    A lldb/tools/lldb-dap/Handler/ConfigurationDoneRequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/ContinueRequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/DisconnectRequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/EvaluateRequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/ExceptionInfoRequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/InitializeRequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/LaunchRequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/RequestHandler.cpp
    A lldb/tools/lldb-dap/Handler/RequestHandler.h
    A lldb/tools/lldb-dap/Handler/RestartRequestHandler.cpp
    M lldb/tools/lldb-dap/lldb-dap.cpp

  Log Message:
  -----------
  [lldb-dap] Refactor request handlers (NFC) (#128262)

Currently, all request handlers are implemented as free functions in
lldb-dap.cpp. That file has grown to over 5000 lines and is starting to
become hard to maintain. This PR moves the request handlers into their
own class (and file), together with their documentation.

This PR migrates about a third of the request handlers and the rest will
be migrated in subsequent commits. I'm merging this in an incomplete
state because almost any lldb-dap change is going to result in merge
conflicts and migrating request handlers one by one is easier to review.



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