summaryrefslogtreecommitdiffstats
path: root/statistics.c
diff options
context:
space:
mode:
authorGravatar Dirk Hohndel <dirk@hohndel.org>2013-04-07 19:50:26 -0700
committerGravatar Dirk Hohndel <dirk@hohndel.org>2013-04-07 20:31:49 -0700
commita70a8898722e1e70ba0ff408414b20fdfce52548 (patch)
treeda68ec64f074eb1984408e43c0866554750160aa /statistics.c
parent387dbe510f14ac5590075544431bbedf1e44fb2f (diff)
downloadsubsurface-a70a8898722e1e70ba0ff408414b20fdfce52548.tar.gz
Fix some of the gcc-4.8 warnings
Most of the warnings are IMHO false positives: e.g.: an enum variable is initialized in a switch statement that has a case for each possible enum value - yet gcc 4.8 warns that it could be used uninitialized; or: two variables are initialized together in the code - second one of them is previously initialized to -1 at declaration time, both are initialized in an if (second one == -1) clause - so they are guaranteed to both be initialized... I did not "fix" those as the code is actually correct. But there are three spots where it catches things that could indeed go wrong (with odd input data in one of them). This commit also adds a check to only call g_type_init() for older versions of glib as in newer ones it is deprecated. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'statistics.c')
-rw-r--r--statistics.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/statistics.c b/statistics.c
index a39799a08..502c06cb4 100644
--- a/statistics.c
+++ b/statistics.c
@@ -668,6 +668,7 @@ static void get_ranges(char *buffer, int size)
}
}
}
+ len = strlen(buffer);
if (first != last) {
if (first + 1 == last)
snprintf(buffer + len, size - len, ", %d", last);