[all-commits] [llvm/llvm-project] b53e75: [lldb-dap] Replace Get{Signed, Unsigned} with GetIn...

Jonas Devlieghere via All-commits all-commits at lists.llvm.org
Wed Mar 5 09:06:36 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b53e75711c284d08b7377385c6f2a037842c0d5b
      https://github.com/llvm/llvm-project/commit/b53e75711c284d08b7377385c6f2a037842c0d5b
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2025-03-05 (Wed, 05 Mar 2025)

  Changed paths:
    M lldb/tools/lldb-dap/DAP.cpp
    M lldb/tools/lldb-dap/Handler/AttachRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/BreakpointLocationsHandler.cpp
    M lldb/tools/lldb-dap/Handler/CompletionsHandler.cpp
    M lldb/tools/lldb-dap/Handler/DataBreakpointInfoRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/DisassembleRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/LocationsRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/ReadMemoryRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/RequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/SetVariableRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/SourceRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/StackTraceRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/StepInRequestHandler.cpp
    M lldb/tools/lldb-dap/Handler/VariablesRequestHandler.cpp
    M lldb/tools/lldb-dap/InstructionBreakpoint.cpp
    M lldb/tools/lldb-dap/JSONUtils.cpp
    M lldb/tools/lldb-dap/JSONUtils.h
    M lldb/tools/lldb-dap/SourceBreakpoint.cpp

  Log Message:
  -----------
  [lldb-dap] Replace Get{Signed,Unsigned} with GetInteger<T> (NFC) (#129823)

Replace Get{Signed,Unsigned} with GetInteger<T> and return std::optional
so you can distinguish between the value not being present and it being
explicitly set to the previous fail_value. All existing uses are
replaced by calling value_or(fail_value).

Continuation of #129818



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