vp8 build warnings in chromium

[GoogleCodeExporter]

E:\b\build\slave\win\build\src\third_party\libvpx\source\libvpx\vp8\encoder\bits
tream.c(175) : warning C4114: same type qualifier used more than once
E:\b\build\slave\win\build\src\third_party\libvpx\source\libvpx\vp8\common\entro
pymode.c(53) : warning C4114: same type qualifier used more than once
E:\b\build\slave\win\build\src\third_party\libvpx\source\libvpx\vp8\common\sad_c
.c(37) : warning C4018: '>' : signed/unsigned mismatch
E:\b\build\slave\win\build\src\third_party\libvpx\source\libvpx\vp8\common\entro
py.c(104) : warning C4114: same type qualifier used more than once

Original issue reported on code.google.com by fbarch...@chromium.org on 1 Oct 2012 at 2:26

[GoogleCodeExporter]

Yaowu can you work with Frank on this? (he is in MTV tomorrow) 

Thanks 

Original comment by albe...@google.com on 11 Oct 2012 at 10:07

• Added labels: Component-libvpx, Mstone-11

[GoogleCodeExporter]

Original comment by albe...@google.com on 11 Oct 2012 at 10:07

• Changed state: Assigned

[GoogleCodeExporter]

These issues were fixed in:
https://gerrit.chromium.org/gerrit/#/c/23159/
https://gerrit.chromium.org/gerrit/#/c/30710/
https://gerrit.chromium.org/gerrit/#/c/31585/

Original comment by ya...@google.com on 26 Nov 2012 at 8:33

[GoogleCodeExporter]

Verified that Chromium build has no warnings when libvpx is updated to "commit 
efb2f92702031cc7adb69bf8561152d2b30152ea"

Original comment by ya...@google.com on 27 Nov 2012 at 5:27

• Changed state: Fixed

[GoogleCodeExporter]

d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\common\vp9_findnearmv.c(85) 
: warning C4028: formal parameter 2 different from declaration
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\common\vp9_findnearmv.c(85) 
: warning C4028: formal parameter 3 different from declaration
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\common\vp9_findnearmv.c(85) 
: warning C4028: formal parameter 4 different from declaration
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\common\vp9_findnearmv.c(85) 
: warning C4028: formal parameter 6 different from declaration
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(709
) : warning C4018: '<' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(714
) : warning C4018: '<' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(742
) : warning C4018: '>=' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(752
) : warning C4018: '<' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(756
) : warning C4018: '<' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(780
) : warning C4018: '>=' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(832
) : warning C4018: '<' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(837
) : warning C4018: '<' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(864
) : warning C4018: '<' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(868
) : warning C4018: '<' : signed/unsigned mismatch
d:\src\svn\src\third_party\libvpx\source\libvpx\vp9\decoder\vp9_decodframe.c(885
) : warning C4018: '>=' : signed/unsigned mismatch

Original comment by tommi@google.com on 18 Jan 2013 at 10:45

[GoogleCodeExporter]

These are new ^^^ as of today afaik. reopen?

Original comment by tommi@google.com on 18 Jan 2013 at 10:46

[GoogleCodeExporter]

@tommi - for now, let's track the newer set of warnings in 
http://crbug.com/171387. 

Original comment by tnakam...@google.com on 24 Jan 2013 at 5:46