Fixed some compiler warnings identified by gcc's -Wextra option.
authorSebastian Harl <sh@tokkee.org>
Tue, 22 Jan 2008 18:11:00 +0000 (19:11 +0100)
committerFlorian Forster <octo@huhu.verplant.org>
Tue, 22 Jan 2008 19:18:38 +0000 (20:18 +0100)
commita2d4be2febb99bf548b991be25f055d37285ddea
treea0bc39ac814910acf89cc5a80b2784b0549c1e54
parent06d0305ee967a12cb541dee87f79c31cac9d1b10
Fixed some compiler warnings identified by gcc's -Wextra option.

The following issues have been addressed:

 * comparison between signed and unsigned - this was found in several places
   throughout the code and has been fixed in various ways
 * missing initializer - an incomplete initializer has been used for two
   struct instances in perl.c
 * unused parameter - when applicable, the parameter has been removed; in
   thirteen cases the parameter is required by different library API's and in
   two cases the parameter was left in place to retain a consistent interface
   within the affected modules; as __attribute__((unused)) is a GNU extension,
   it has not been used to document those exceptions

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>
18 files changed:
src/apcups.c
src/battery.c
src/collectd.c
src/collectdmon.c
src/common.c
src/cpufreq.c
src/disk.c
src/iptables.c
src/irq.c
src/network.c
src/ntpd.c
src/perl.c
src/plugin.c
src/plugin.h
src/types_list.c
src/unixsock.c
src/utils_avltree.c
src/utils_dns.c