[all-commits] [llvm/llvm-project] f195f4: [lldb] Add comma separator in watchpoint descripti...

[David Spickett via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f195f4d45dc253192270a4be3f806f2913a071d1
      https://github.com/llvm/llvm-project/commit/f195f4d45dc253192270a4be3f806f2913a071d1
  Author: David Spickett <david.spickett at arm.com>
  Date:   2026-07-14 (Tue, 14 Jul 2026)

  Changed paths:
    M lldb/bindings/interface/SBTargetDocstrings.i
    M lldb/docs/use/tutorial.md
    M lldb/source/Breakpoint/Watchpoint.cpp
    M lldb/test/Shell/Register/x86-db-read.test

  Log Message:
  -----------
  [lldb] Add comma separator in watchpoint description (#209176)

I find it easier to read this way, especially the last "state = enabled type = m" which looks like "enabled type = m" at a glance:
Watchpoint 1: addr = 0xaaaaaaab1018 size = 20 state = enabled type = m

With commas:
Watchpoint 1: addr = 0xaaaaaaab1018, size = 20, state = enabled, type = m

Code breakpoints include commas already:
1: name = 'main', locations = 1, resolved = 1, hit count = 1
1.1: where = test.o`main at test.c:13:15, address = 0x0000aaaaaaaa0714, resolved, hit count = 1

Used the regex "Watchpoint [0-9]+:" to find tests and docs that needed
updating.



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