Signed/unsigned warning in decodemv.c

Chen-tao / webm

Automatically exported from code.google.com/p/webm

0 stars 0 forks source link

Signed/unsigned warning in decodemv.c #475

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

The webrtc standalone build uses:
http://git.chromium.org/webm/libvpx.git@c522217d

Warnings-as-errors is about to be enabled on Windows by default. This trips on 
the following warning:
2>source\libvpx\vp8\decoder\decodemv.c(647): error C2220: warning treated as 
error - no 'object' file generated
2>source\libvpx\vp8\decoder\decodemv.c(647): warning C4018: '<' : 
signed/unsigned mismatch

Original issue reported on code.google.com by ajm@google.com on 27 Aug 2012 at 11:31

GoogleCodeExporter commented 9 years ago

Yaowu, can you double check this one too?    Seems we missed it.

Original comment by jimbankoski@google.com on 27 Aug 2012 at 11:41

Changed state: Assigned

GoogleCodeExporter commented 9 years ago

[deleted comment]

GoogleCodeExporter commented 9 years ago

Original comment by ya...@google.com on 28 Aug 2012 at 1:37

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Now addressed in https://gerrit.chromium.org/gerrit/#/c/31585/

The line causing the compiler warning was hidden behind 
CONFIG_ERROR_CONCEALMENT, therefore was missed in the previous patch.

Original comment by ya...@google.com on 28 Aug 2012 at 1:39