From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=36734 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Oq6lc-0004yJ-0z for qemu-devel@nongnu.org; Mon, 30 Aug 2010 11:59:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1Oq6la-0007YR-RR for qemu-devel@nongnu.org; Mon, 30 Aug 2010 11:59:27 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60023) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1Oq6la-0007YI-J0 for qemu-devel@nongnu.org; Mon, 30 Aug 2010 11:59:26 -0400 From: Jes.Sorensen@redhat.com Date: Mon, 30 Aug 2010 17:59:06 +0200 Message-Id: <1283183960-28404-1-git-send-email-Jes.Sorensen@redhat.com> Subject: [Qemu-devel] [PATCH 00/14] gcc extra warning fixes v2 List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, Jes.Sorensen@redhat.com From: Jes Sorensen 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. v2 fixes the vnc change as pointed out by Anthony. 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, change to ssize_t to handle errors from tight_compress_data() 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