From: Yann Collet Date: Sat, 9 Apr 2016 14:17:18 +0000 (+0200) Subject: fixed minor warnings when compiling with `gcc -g` (aka `-std=gnu90`) X-Git-Tag: v0.6.0^2~17^2~3 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9990986bdc735ce18a0484e6136db3474d28b490;p=thirdparty%2Fzstd.git fixed minor warnings when compiling with `gcc -g` (aka `-std=gnu90`) --- diff --git a/lib/legacy/zstd_v05.c b/lib/legacy/zstd_v05.c index d1fba48bb..d50724793 100644 --- a/lib/legacy/zstd_v05.c +++ b/lib/legacy/zstd_v05.c @@ -3952,7 +3952,7 @@ static void ZSTDv05_decodeSequence(seq_t* seq, seqState_t* seqState) } -FORCE_INLINE size_t ZSTDv05_execSequence(BYTE* op, +static size_t ZSTDv05_execSequence(BYTE* op, BYTE* const oend, seq_t sequence, const BYTE** litPtr, const BYTE* const litLimit_8, const BYTE* const base, const BYTE* const vBase, const BYTE* const dictEnd) diff --git a/programs/bench.c b/programs/bench.c index cea9634c2..37c1e9f77 100644 --- a/programs/bench.c +++ b/programs/bench.c @@ -1,6 +1,6 @@ /* - bench.c - Demo module to benchmark open-source compression algorithms - Copyright (C) Yann Collet 2012-2015 + bench.c - open-source compression benchmark module + Copyright (C) Yann Collet 2012-2016 GPL v2 License @@ -19,8 +19,8 @@ 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. You can contact the author at : + - zstd homepage : http://www.zstd.net - zstd source repository : https://github.com/Cyan4973/zstd - - ztsd public forum : https://groups.google.com/forum/#!forum/lz4c */ /* ************************************** @@ -44,13 +44,13 @@ /* ************************************* * Includes ***************************************/ -#define _POSIX_C_SOURCE 199309L /* before - needed for nanosleep() */ +#define _POSIX_C_SOURCE 199309L /* before - needed for nanosleep() */ #include /* malloc, free */ #include /* memset */ #include /* fprintf, fopen, ftello64 */ #include /* stat64 */ #include /* stat64 */ -#include /* clock_t, nanosleep, clock, CLOCKS_PER_SEC */ +#include /* clock_t, nanosleep, clock, CLOCKS_PER_SEC */ /* sleep : posix - windows - others */ #if !defined(_WIN32) && (defined(__unix__) || defined(__unix) || (defined(__APPLE__) && defined(__MACH__))) @@ -97,7 +97,6 @@ #include "xxhash.h" - /* ************************************* * Compiler specifics ***************************************/ @@ -105,8 +104,12 @@ # define S_ISREG(x) (((x) & S_IFMT) == S_IFREG) #endif -#ifdef _MSC_VER -#define snprintf sprintf_s +#if defined(_MSC_VER) +# define snprintf sprintf_s +#elif defined (__cplusplus) || (defined (__STDC_VERSION__) && (__STDC_VERSION__ >= 199901L)) + /* part of */ +#else + extern int snprintf (char* s, size_t maxlen, const char* format, ...); /* not declared in when C version < c99 */ #endif @@ -187,7 +190,7 @@ void BMK_SetBlockSize(size_t blockSize) static U64 BMK_clockSpan( BMK_time_t clockStart, BMK_time_t ticksPerSecond ) { BMK_time_t clockEnd; - + (void)ticksPerSecond; BMK_getTime(clockEnd); return BMK_getSpanTimeMicro(ticksPerSecond, clockStart, clockEnd); @@ -230,7 +233,7 @@ typedef struct double dSpeed; } benchResult_t; - + #define MIN(a,b) ((a)<(b) ? (a) : (b)) #define MAX(a,b) ((a)>(b) ? (a) : (b)) @@ -406,7 +409,7 @@ static int BMK_benchMem(const void* srcBuffer, size_t srcSize, if (crcOrig == crcCheck) { result->ratio = ratio; result->cSize = cSize; - result->cSpeed = (double)srcSize / fastestC; + result->cSpeed = (double)srcSize / fastestC; result->dSpeed = (double)srcSize / fastestD; } DISPLAYLEVEL(2, "%2i#\n", cLevel); @@ -463,7 +466,7 @@ static void BMK_benchCLevel(void* srcBuffer, size_t benchedSize, if (cLevelLast < cLevel) cLevelLast = cLevel; - for (l=cLevel; l <= cLevelLast; l++) { + for (l=cLevel; l <= cLevelLast; l++) { BMK_benchMem(srcBuffer, benchedSize, displayName, l, fileSizes, nbFiles, @@ -477,15 +480,13 @@ static void BMK_benchCLevel(void* srcBuffer, size_t benchedSize, total.cSpeed += result.cSpeed; total.dSpeed += result.dSpeed; total.ratio += result.ratio; - } - } - if (g_displayLevel == 1 && cLevelLast > cLevel) - { + } } + if (g_displayLevel == 1 && cLevelLast > cLevel) { total.cSize /= 1+cLevelLast-cLevel; total.cSpeed /= 1+cLevelLast-cLevel; total.dSpeed /= 1+cLevelLast-cLevel; total.ratio /= 1+cLevelLast-cLevel; - DISPLAY("avg%11i (%5.3f) %6.1f MB/s %6.1f MB/s %s\n", (int)total.cSize, total.ratio, total.cSpeed, total.dSpeed, displayName); + DISPLAY("avg%11i (%5.3f) %6.1f MB/s %6.1f MB/s %s\n", (int)total.cSize, total.ratio, total.cSpeed, total.dSpeed, displayName); } }