From: Jes Sorensen <jes.soren...@redhat.com> Hi,
I started building QEMU with some more aggressive error flags to see what dropped out. I started fixing up some of the cases, removing unused arguments to functions, comparisons of unsigned types against negative values etc. and a few other minor changes to avoid compiler warnings. Set of 14 patches following. Cheers, Jes Jes Sorensen (14): Remove unused argument for nbd_client() Respect return value from nbd_client() Fix repeated typo: was "end if list" instead of "end of list" Zero initialize timespec struct explicitly Remove unused argument for check_for_block_signature() Remove unused argument for encrypt_sectors() Remove unused argument for get_whole_cluster() Remove unused argument for qcow2_encrypt_sectors() Remove unused arguments for add_aio_request() and free_aio_req() Zero json struct with memset() instea of = {} to keep compiler happy. Remove unused function arguments size_t is unsigned, so (foo >= 0) is always true Change DPRINTF() to do{}while(0) to avoid compiler warning load_multiboot(): get_image_size() returns int block/qcow.c | 13 ++++++------- block/qcow2-cluster.c | 17 ++++++++--------- block/qcow2.c | 10 +++++----- block/qcow2.h | 7 +++---- block/raw.c | 4 ++-- block/sheepdog.c | 20 ++++++++++---------- block/vmdk.c | 4 ++-- hw/multiboot.c | 2 +- json-parser.c | 39 +++++++++++++++++++-------------------- linux-aio.c | 2 +- nbd.c | 4 ++-- nbd.h | 2 +- qemu-config.c | 12 ++++++------ qemu-nbd.c | 5 ++++- qjson.c | 3 ++- slirp/bootp.c | 2 +- ui/vnc-enc-tight.c | 8 ++++---- 17 files changed, 77 insertions(+), 77 deletions(-) -- 1.7.2.2