Vladimir Bespalov : winedbg: Fix enum value compilation warning.

Alexandre Julliard julliard at winehq.org
Mon May 14 06:54:26 CDT 2018


Module: wine
Branch: stable
Commit: d2ad933a28d665df1852aa1ad294f4628b9a11a3
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=d2ad933a28d665df1852aa1ad294f4628b9a11a3

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>
(cherry picked from commit 57ccf8221fdabfb90c4117b34478919e21e72858)
Signed-off-by: Michael Stefaniuc <mstefani 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 a87fe02..cfa9fe3 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;
 };
@@ -1514,7 +1514,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;
         }
     }
@@ -2123,7 +2123,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;
@@ -2164,7 +2164,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