Vladimir Bespalov : winedbg: Fix enum value compilation warning.

Alexandre Julliard julliard at winehq.org
Wed Feb 7 15:24:27 CST 2018


Module: wine
Branch: master
Commit: 57ccf8221fdabfb90c4117b34478919e21e72858
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=57ccf8221fdabfb90c4117b34478919e21e72858

Author: Vladimir Bespalov <vlad.botanic at gmail.com>
Date:   Fri Feb  2 04:11:26 2018 -0800

winedbg: Fix enum value compilation warning.

Signed-off-by: Vlad Bespalov <vlad.botanic at gmail.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 programs/winedbg/be_cpu.h   | 2 +-
 programs/winedbg/gdbproxy.c | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/programs/winedbg/be_cpu.h b/programs/winedbg/be_cpu.h
index 889f638..d62b517 100644
--- a/programs/winedbg/be_cpu.h
+++ b/programs/winedbg/be_cpu.h
@@ -20,7 +20,7 @@
 
 enum be_cpu_addr {be_cpu_addr_pc, be_cpu_addr_stack, be_cpu_addr_frame};
 enum be_xpoint_type {be_xpoint_break, be_xpoint_watch_exec, be_xpoint_watch_read,
-                     be_xpoint_watch_write};
+                     be_xpoint_watch_write, be_xpoint_free=-1};
 struct backend_cpu
 {
     const DWORD         machine;
diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c
index 7b9a476..5fa4883 100644
--- a/programs/winedbg/gdbproxy.c
+++ b/programs/winedbg/gdbproxy.c
@@ -74,7 +74,7 @@
 
 struct gdb_ctx_Xpoint
 {
-    enum be_xpoint_type         type;   /* -1 means free */
+    enum be_xpoint_type         type;   /* (-1) == be_xpoint_free means free */
     void*                       addr;
     unsigned long               val;
 };
@@ -1583,7 +1583,7 @@ static BOOL read_memory(struct gdb_context *gdbctx, char *addr, char *buffer, SI
         {
             char *xpt_addr = xpt->addr;
 
-            if (xpt->type != -1 && xpt_addr >= addr && xpt_addr < addr + blk_len)
+            if (xpt->type != be_xpoint_free && xpt_addr >= addr && xpt_addr < addr + blk_len)
                 buffer[xpt_addr - addr] = xpt->val;
         }
     }
@@ -2192,7 +2192,7 @@ static enum packet_return packet_remove_breakpoint(struct gdb_context* gdbctx)
                                       gdbctx->process->process_io, &gdbctx->context,
                                       t, xpt->addr, xpt->val, len))
             {
-                xpt->type = -1;
+                xpt->type = be_xpoint_free;
                 return packet_ok;
             }
             break;
@@ -2233,7 +2233,7 @@ static enum packet_return packet_set_breakpoint(struct gdb_context* gdbctx)
     /* really set the Xpoint */
     for (xpt = &gdbctx->Xpoints[NUM_XPOINT - 1]; xpt >= gdbctx->Xpoints; xpt--)
     {
-        if (xpt->type == -1)
+        if (xpt->type == be_xpoint_free)
         {
             if (be_cpu->insert_Xpoint(gdbctx->process->handle,
                                       gdbctx->process->process_io, &gdbctx->context, 




More information about the wine-cvs mailing list