#92 Warnings in building SPTK-3.10

[Akira Tamamori]

Following warnings were appeared in building SPTK-3.10.
I think stdlib.h should be included in _lmadf.c, and ctype.h should be included in lmadf.c .
I hope that this information helps you perform debug.

agcep.c: In function ‘main’:
agcep.c:175:38: warning: unused variable ‘mu’ [-Wunused-variable]
        *c, *cc, *avec, tau = TAU, x, mu, gamma;

../bin/lmadf/_lmadf.c: In function ‘cascade_lmadf’:
../bin/lmadf/_lmadf.c:125:20: warning: implicit declaration of function ‘abs’ [-Wimplicit-function-declaration]
       block_end += abs(block_size[i]);

lmadf.c: In function ‘main’:
lmadf.c:221:21: warning: implicit declaration of function ‘isdigit’ [-Wimplicit-function-declaration]
                if (!isdigit(**argv)

snack/jkGetF0.c: In function ‘get_cand’:
snack/jkGetF0.c:546:20: warning: variable ‘maxl’ set but not used [-Wunused-but-set-variable]
   int start, ncan, maxl;

snack/jkGetF0.c: In function ‘rapt’:
snack/jkGetF0.c:1865:14: warning: variable ‘start_time’ set but not used [-Wunused-but-set-variable]
   double sf, start_time;

snack/sigproc.c: In function ‘crossf’:
snack/sigproc.c:554:21: warning: variable ‘maxsize’ set but not used [-Wunused-but-set-variable]
   int sizei, sizeo, maxsize;

snack/sigproc.c:554:14: warning: variable ‘sizeo’ set but not used [-Wunused-but-set-variable]
   int sizei, sizeo, maxsize;

snack/sigproc.c:554:7: warning: variable ‘sizei’ set but not used [-Wunused-but-set-variable]
   int sizei, sizeo, maxsize;

fig1.c: In function ‘sleng’:
fig1.c:232:30: warning: unused parameter ‘h’ [-Wunused-parameter]
 double sleng(char *p, double h, double w)

fileio.c: In function ‘fwritef’:
fileio.c:130:39: warning: unused parameter ‘size’ [-Wunused-parameter]
 int fwritef(double *ptr, const size_t size, const int nitems, FILE * fp)

fileio.c: In function ‘freadf’:
fileio.c:150:38: warning: unused parameter ‘size’ [-Wunused-parameter]
 int freadf(double *ptr, const size_t size, const int nitems, FILE * fp)

x2x.c: In function ‘x2x’:
x2x.c:683:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
          if (xl > ULONG_MAX || xl < 0 || xul > ULONG_MAX || xd > ULONG_MAX

x2x.c:688:36: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
          xl = ((xl < 0) ? 0 : ((xl > ULONG_MAX) ? ULONG_MAX : xl));

x2x.c:688:61: warning: signed and unsigned type in conditional expression [-Wsign-compare]
          xl = ((xl < 0) ? 0 : ((xl > ULONG_MAX) ? ULONG_MAX : xl));

x2x.c:823:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
          if (xl > ULONG_MAX || xl < 0 || xul > ULONG_MAX || xd > ULONG_MAX

hts_engine_API/HTS_pstream.c: In function ‘HTS_PStream_calc_wuw_and_wum’:
hts_engine_API/HTS_pstream.c:94:60: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
             if (((int) t + shift >= 0) && ((int) t + shift < pst->length) && (pst->win_coefficient[i][-shift] != 0.0)) {

[fujishita taichi]

Sorry for the late response.

As you said, the warnings are cleared when the header files are included. This problem will be fixed in the next stable release.

Thank you for your kind report!