[all-commits] [llvm/llvm-project] 02f024: [LLDB-DAP] SBDebugger don't prefix title on progre...

[Jacob Lalonde via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 02f024ca97403e8dff55ca4feebe78009d9ea8f3
      https://github.com/llvm/llvm-project/commit/02f024ca97403e8dff55ca4feebe78009d9ea8f3
  Author: Jacob Lalonde <jalalonde at fb.com>
  Date:   2025-03-05 (Wed, 05 Mar 2025)

  Changed paths:
    M lldb/bindings/interface/SBProgressDocstrings.i
    M lldb/test/API/tools/lldb-dap/progress/Progress_emitter.py
    M lldb/test/API/tools/lldb-dap/progress/TestDAP_Progress.py
    M lldb/tools/lldb-dap/Handler/InitializeRequestHandler.cpp

  Log Message:
  -----------
  [LLDB-DAP] SBDebugger don't prefix title on progress updates (#124648)

In my last DAP patch (#123837), we piped the DAP update message into the
update event. However, we had the title embedded into the update
message. This makes sense for progress Start, but makes the update
message look pretty wonky.


![image](https://github.com/user-attachments/assets/9f6083d1-fc50-455c-a1a7-a2f9bdaba22e)

Instead, we only use the title when it's the first message, removing the
duplicate title problem.

![image](https://github.com/user-attachments/assets/ee7aefd1-1852-46f7-94bc-84b8faef6dac)



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