This is an automated email from Gerrit.

"Marek Vrbka <marek.vr...@codasip.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7909

-- gerrit

commit 86d1d408147f640d488af133004e1abe43b848b6
Author: Marek Vrbka <marek.vr...@codasip.com>
Date:   Mon Sep 25 10:00:42 2023 +0200

    target: Change the watchpoint type print from number to letter
    
    Previously, when listing the watchpoints, OpenOCD printed
    numbers 0, 1 and 2 representing READ, WRITE and ACCESS type
    watchpoints.
    
    This patch changes it to 'r', 'w' and 'a'. This increases the
    clarity as what type the watchpoint actually is.
    
    Change-Id: I9eac72dfd0bb2a9596a5b0c080a3f584556ed599
    Signed-off-by: Marek Vrbka <marek.vr...@codasip.com>

diff --git a/src/target/target.c b/src/target/target.c
index 1219743750..9be154cb4d 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -4071,11 +4071,11 @@ COMMAND_HANDLER(handle_wp_command)
                while (watchpoint) {
                        command_print(CMD, "address: " TARGET_ADDR_FMT
                                        ", len: 0x%8.8" PRIx32
-                                       ", r/w/a: %i, value: 0x%8.8" PRIx64
+                                       ", r/w/a: %c, value: 0x%8.8" PRIx64
                                        ", mask: 0x%8.8" PRIx64,
                                        watchpoint->address,
                                        watchpoint->length,
-                                       (int)watchpoint->rw,
+                                       (watchpoint->rw == WPT_READ ? 'r' : 
(watchpoint->rw == WPT_WRITE ? 'w' : 'a')),
                                        watchpoint->value,
                                        watchpoint->mask);
                        watchpoint = watchpoint->next;

-- 

Reply via email to