[all-commits] [llvm/llvm-project] 1041d5: [lldb-dap] Updating the 'next' request handler use...

John Harrison via All-commits all-commits at lists.llvm.org
Wed Apr 23 10:18:53 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1041d54bd4f693c1ac03077680ece67e03c99e22
      https://github.com/llvm/llvm-project/commit/1041d54bd4f693c1ac03077680ece67e03c99e22
  Author: John Harrison <harjohn at google.com>
  Date:   2025-04-23 (Wed, 23 Apr 2025)

  Changed paths:
    M lldb/tools/lldb-dap/DAP.cpp
    M lldb/tools/lldb-dap/DAP.h
    M lldb/tools/lldb-dap/Handler/CancelRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/DisconnectRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/NextRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/RequestHandler.h
    M lldb/tools/lldb-dap/Protocol/ProtocolBase.h
    M lldb/tools/lldb-dap/Protocol/ProtocolRequests.cpp
    M lldb/tools/lldb-dap/Protocol/ProtocolRequests.h
    M lldb/tools/lldb-dap/Protocol/ProtocolTypes.cpp
    M lldb/tools/lldb-dap/Protocol/ProtocolTypes.h
    M lldb/tools/lldb-dap/Transport.cpp

  Log Message:
  -----------
  [lldb-dap] Updating the 'next' request handler use well structured types (#136642)

This updates the 'next' request to use well structured types. While
working on this I also simplified the 'RequestHandler' implementation to
better handle void responses by allowing requests to return a
'llvm::Error' instead of an 'llvm::Expected<std::monostate>'. This makes
it easier to write and understand request handles that have simple ack
responses.



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


More information about the All-commits mailing list