search

Chen-tao / webm

Automatically exported from code.google.com/p/webm
0 stars 0 forks source link

Warnings building on Windows #466

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
In Chromium @149334:

[2539->8456/11012 ~17] CC 
obj\third_party\libvpx\source\libvpx\vp8\common\libvpx.entropy.obj
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\common\entropy.c(104) : 
warning C4114: same type qualifier used more than once
[2535->8460/11012 ~17] CC 
obj\third_party\libvpx\source\libvpx\vp8\common\libvpx.entropymode.obj
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\common\entropymode.c(53) : 
warning C4114: same type qualifier used more than once
[2528->8467/11012 ~17] CC 
obj\third_party\libvpx\source\libvpx\vp8\common\libvpx.sad_c.obj
f:\src\old\src\(37) : warning C4018: '>' : signed/unsigned mismatch
[2504->8491/11012 ~17] CC 
obj\third_party\libvpx\source\libvpx\vp8\encoder\libvpx.bitstream.obj
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\bitstream.c(175) : 
warning C4114: same type qualifier used more than once

C4114 seems a bit silly, perhaps just disable that one.
sad_c.c just needs a changed type.

Original issue reported on code.google.com by scottmg@chromium.org on 1 Aug 2012 at 8:18

GoogleCodeExporter commented 9 years ago 
Just checked ToT from http://git.chromium.org/webm/libvpx.git. All the above 
appear fixed, but others have been created. :(

f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1946) : 
warning C4028: formal parameter 5 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1954) : 
warning C4028: formal parameter 3 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1956) : 
warning C4028: formal parameter 5 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1964) : 
warning C4028: formal parameter 3 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1966) : 
warning C4028: formal parameter 5 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1974) : 
warning C4028: formal parameter 3 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1976) : 
warning C4028: formal parameter 5 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1984) : 
warning C4028: formal parameter 3 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1986) : 
warning C4028: formal parameter 5 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(1994) : 
warning C4028: formal parameter 3 different from
declaration
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(5430) : 
warning C4018: '>' : signed/unsigned mismatch
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(5430) : 
warning C4018: '>' : signed/unsigned mismatch
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(5431) : 
warning C4018: '>' : signed/unsigned mismatch
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(5431) : 
warning C4018: '>' : signed/unsigned mismatch
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(5435) : 
warning C4018: '>' : signed/unsigned mismatch
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(5435) : 
warning C4018: '>' : signed/unsigned mismatch
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(5436) : 
warning C4018: '>' : signed/unsigned mismatch
f:\src\old\src\third_party\libvpx\source\libvpx\vp8\encoder\onyx_if.c(5436) : 
warning C4018: '>' : signed/unsigned mismatch

Original comment by scottmg@chromium.org on 1 Aug 2012 at 8:27

GoogleCodeExporter commented 9 years ago 
This bug has started, once the code is committed will be mark as fixed. 

https://gerrit.chromium.org/gerrit/#/c/30710/

Original comment by albe...@google.com on 17 Aug 2012 at 8:42

GoogleCodeExporter commented 9 years ago 
https://gerrit.chromium.org/gerrit/#/c/30710/ is now merged, the issue is fixed.

Original comment by ya...@google.com on 20 Aug 2012 at 10:30