diff options
author | Blue Swirl <blauwirbel@gmail.com> | 2010-04-18 08:45:03 +0000 |
---|---|---|
committer | Blue Swirl <blauwirbel@gmail.com> | 2010-04-18 08:46:35 +0000 |
commit | 3c83eb4f17ffa981cb60b965087769b808062a3a (patch) | |
tree | 45935cc30e5126dec2fdf7bd724f6c67029f47e0 | |
parent | 606257c6f2dbbaa63f462e557f15a5f7364967a6 (diff) |
Fix harmless if statements with empty body, spotted by clang
These clang errors are harmless but worth fixing:
CC libhw64/fdc.o
/src/qemu/hw/fdc.c:998:74: error: if statement has empty body [-Wempty-body]
FLOPPY_DPRINTF("Floppy digital input register: 0x%02x\n", retval);
CC libhw64/cuda.o
/src/qemu/hw/cuda.c:320:66: error: if statement has empty body [-Wempty-body]
CUDA_DPRINTF("read: reg=0x%x val=%02x\n", (int)addr, val);
Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
-rw-r--r-- | hw/cuda.c | 4 | ||||
-rw-r--r-- | hw/fdc.c | 3 |
2 files changed, 5 insertions, 2 deletions
@@ -316,8 +316,10 @@ static uint32_t cuda_readb(void *opaque, target_phys_addr_t addr) val = s->anh; break; } - if (addr != 13 || val != 0) + if (addr != 13 || val != 0) { CUDA_DPRINTF("read: reg=0x%x val=%02x\n", (int)addr, val); + } + return val; } @@ -994,8 +994,9 @@ static uint32_t fdctrl_read_dir(FDCtrl *fdctrl) #endif ) retval |= FD_DIR_DSKCHG; - if (retval != 0) + if (retval != 0) { FLOPPY_DPRINTF("Floppy digital input register: 0x%02x\n", retval); + } return retval; } |