[all-commits] [llvm/llvm-project] a939a9: [LLDB-DAP] Send Progress update message over DAP (...

Jacob Lalonde via All-commits all-commits at lists.llvm.org
Wed Jan 22 15:49:34 PST 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a939a9fd53d98f33b94f9121646d5906a2b9f598
      https://github.com/llvm/llvm-project/commit/a939a9fd53d98f33b94f9121646d5906a2b9f598
  Author: Jacob Lalonde <jalalonde at fb.com>
  Date:   2025-01-22 (Wed, 22 Jan 2025)

  Changed paths:
    A lldb/test/API/tools/lldb-dap/progress/Makefile
    A lldb/test/API/tools/lldb-dap/progress/Progress_emitter.py
    A lldb/test/API/tools/lldb-dap/progress/TestDAP_Progress.py
    A lldb/test/API/tools/lldb-dap/progress/main.cpp
    M lldb/tools/lldb-dap/ProgressEvent.cpp
    M lldb/tools/lldb-dap/ProgressEvent.h

  Log Message:
  -----------
  [LLDB-DAP] Send Progress update message over DAP (#123837)

When testing my SBProgress DAP PR (#123826), I noticed Progress update
messages aren't sent over DAP. This patch adds the lldb progress event's
message to the body when sent over DAP.

Before 

![image](https://github.com/user-attachments/assets/404adaa8-b784-4f23-895f-cd3625fdafad)


Now

![image](https://github.com/user-attachments/assets/eb1c3235-0936-4e36-96e5-0a0ee60dabb8)

Tested with my [progress tester
command](https://gist.github.com/Jlalond/48d85e75a91f7a137e3142e6a13d0947),
testing 10 events 5 seconds apart 1-10



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