1371536 - Suppress more MSVC warnings in media/libspeex_resampler

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, enhancement)

Description

[Masatoshi Kimura [:emk]]

3:30.15 warning: media/libspeex_resampler/src/resample.c:687 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.15 warning: media/libspeex_resampler/src/resample.c:814 [C4101] 'i': ローカル変数は 1 度も使われていません。
 3:30.15 warning: media/libspeex_resampler/src/resample.c:960 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.15 warning: media/libspeex_resampler/src/resample.c:963 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.15 warning: media/libspeex_resampler/src/resample.c:1016 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.16 warning: media/libspeex_resampler/src/resample.c:1023 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.16 warning: media/libspeex_resampler/src/resample.c:1033 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.16 warning: media/libspeex_resampler/src/resample_sse.c:47 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.16 warning: media/libspeex_resampler/src/resample_sse.c:64 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.16 warning: media/libspeex_resampler/src/resample_sse.c:86 [C4018] '<': signed と unsigned の数値を比較しようとしました。
 3:30.16 warning: media/libspeex_resampler/src/resample_sse.c:112 [C4018] '<': signed と unsigned の数値を比較しようとしました。

Comment 1

[Masatoshi Kimura [:emk]]

Attachment: Bug 1371536 - Suppress more MSVC warnings in media/libspeex_resampler.

Review commit: https://reviewboard.mozilla.org/r/147454/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/147454/

Comment 2

[Paul Adenot (:padenot)]

Comment on attachment 8876012 [details]
Bug 1371536 - Suppress more MSVC warnings in media/libspeex_resampler.

https://reviewboard.mozilla.org/r/147454/#review152282

Thanks!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c7979b14d998
Suppress more MSVC warnings in media/libspeex_resampler. r=padenot

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/c7979b14d998

Comment 5

[Ionuț Goldan [:igoldan]]

This reduced lots of compiler errors for the Windows platform. Thanks!

== Change summary for alert #7210 (as of June 12 2017 13:23 UTC) ==

Improvements:

  8%  compiler warnings summary windowsxp opt      145.00 -> 134.00
  8%  compiler warnings summary windows2012-32 opt 145.00 -> 134.00
  7%  compiler warnings summary windows2012-32-noopt debug 151.00 -> 140.00
  7%  compiler warnings summary windowsxp debug    151.00 -> 140.00
  7%  compiler warnings summary windows2012-32 debug 151.00 -> 140.00
  7%  compiler warnings summary windows2012-32 pgo 156.50 -> 145.42
  7%  compiler warnings summary windows8-64 opt    164.00 -> 153.00
  7%  compiler warnings summary windows2012-64 opt 164.00 -> 153.00
  6%  compiler warnings summary windows2012-64-noopt debug 171.00 -> 160.00
  6%  compiler warnings summary windows8-64 debug  171.00 -> 160.00
  6%  compiler warnings summary windows2012-64 debug 171.00 -> 160.00
  6%  compiler warnings summary windows2012-64 pgo 175.25 -> 164.33

For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=7210