[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 00/14] gcc extra warning fixes v2
From: |
Jes . Sorensen |
Subject: |
[Qemu-devel] [PATCH 00/14] gcc extra warning fixes v2 |
Date: |
Mon, 30 Aug 2010 17:59:06 +0200 |
From: Jes Sorensen <address@hidden>
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
- [Qemu-devel] [PATCH 00/14] gcc extra warning fixes v2,
Jes . Sorensen <=
- [Qemu-devel] [PATCH 13/14] Change DPRINTF() to do{}while(0) to avoid compiler warning, Jes . Sorensen, 2010/08/30
- [Qemu-devel] [PATCH 05/14] Remove unused argument for check_for_block_signature(), Jes . Sorensen, 2010/08/30
- [Qemu-devel] Re: [PATCH 05/14] Remove unused argument for check_for_block_signature(), Anthony Liguori, 2010/08/30
- [Qemu-devel] Re: [PATCH 05/14] Remove unused argument for check_for_block_signature(), Paolo Bonzini, 2010/08/30
- [Qemu-devel] Re: [PATCH 05/14] Remove unused argument for check_for_block_signature(), Jes Sorensen, 2010/08/30
- [Qemu-devel] Re: [PATCH 05/14] Remove unused argument for check_for_block_signature(), Anthony Liguori, 2010/08/30
- Re: [Qemu-devel] Re: [PATCH 05/14] Remove unused argument for check_for_block_signature(), Richard Henderson, 2010/08/30
- [Qemu-devel] Re: [PATCH 05/14] Remove unused argument for check_for_block_signature(), Jes Sorensen, 2010/08/31
[Qemu-devel] [PATCH 11/14] Remove unused function arguments, Jes . Sorensen, 2010/08/30
[Qemu-devel] [PATCH 10/14] Zero json struct with memset() instea of = {} to keep compiler happy., Jes . Sorensen, 2010/08/30