[gdb] Fix a compiler warning that shows up only when assertions are enabled
authorMichael Brown <mcb30@etherboot.org>
Mon, 29 Sep 2008 00:00:14 +0000 (01:00 +0100)
committerMichael Brown <mcb30@etherboot.org>
Mon, 29 Sep 2008 00:00:14 +0000 (01:00 +0100)
gcc should (I think) be warning about this anyway, but seems to do so
only when assertions are enabled for this object.

src/arch/i386/core/gdbmach.c

index 26fab60..5e72e4d 100644 (file)
@@ -60,10 +60,10 @@ static struct hwbp *gdbmach_find_hwbp ( int type, unsigned long addr, size_t len
 }
 
 static void gdbmach_commit_hwbp ( struct hwbp *bp ) {
-       int regnum = bp - hwbps;
+       unsigned int regnum = bp - hwbps;
 
        /* Set breakpoint address */
-       assert ( regnum >= 0 && regnum < sizeof hwbps / sizeof hwbps [ 0 ] );
+       assert ( regnum < ( sizeof hwbps / sizeof hwbps [ 0 ] ) );
        switch ( regnum ) {
                case 0:
                        __asm__ __volatile__ ( "movl %0, %%dr0\n" : : "r" ( bp->addr ) );