SDL_audiolib icon indicating copy to clipboard operation
SDL_audiolib copied to clipboard

Compiler warnings

Open AJenbo opened this issue 2 years ago • 2 comments

These are now the only compiler warnings left when building DevilutionX, would be great if we can handle them so we can start checking for compiler warnings in the CI :)

In file included from /home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample.c:100:
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample_sse.h:45:14: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
   for (i=0;i<len;i+=8)
            ~^~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample_sse.h:62:12: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
  for(i=0;i<len;i+=2)
          ~^~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample_sse.h:84:14: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
   for (i=0;i<len;i+=8)
            ~^~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample_sse.h:110:12: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
  for(i=0;i<len;i+=2)
          ~^~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample.c:674:20: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
         for (j=0;j<st->filt_len;j++)
                  ~^~~~~~~~~~~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample.c:946:21: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
           for(j=0;j<ichunk;++j)
                   ~^~~~~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample.c:949:20: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
          for(j=0;j<ichunk;++j)
                  ~^~~~~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample.c:1001:19: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
         for(j=0;j<ichunk;++j)
                 ~^~~~~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample.c:1008:19: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
         for(j=0;j<ichunk;++j)
                 ~^~~~~~~
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/3rdparty/speex_resampler/resample.c:1018:16: warning: comparison of integers of different signs: 'int' and 'unsigned int' [-Wsign-compare]
     for (j=0;j<ochunk+omagic;++j)
              ~^~~~~~~~~~~~~~
[ 46%] Built target StormLib
10 warnings generated.
/home/ajenbo/code/diablo/devilutionX/buildclang/_deps/sdl_audiolib-src/src/Decoder.cpp:54:10: warning: unused variable 'tryDecoder' [-Wunused-variable]
    auto tryDecoder = [rwops, &rewindRwops](auto dec) {
         ^

AJenbo avatar Jul 17 '21 15:07 AJenbo

I pushed a fix for the Decoder.cpp warning. As for the resampler, you'd have to bug upstream about it:

https://gitlab.xiph.org/xiph/opus-tools

:-P

realnc avatar Jul 18 '21 08:07 realnc

https://gitlab.xiph.org/xiph/opus-tools/-/issues/2317

AJenbo avatar Jul 18 '21 14:07 AJenbo