teasame / webp

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

webp 0.1.3 build warnings on Visual Studio 2008 #110

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
third_party\libwebp\files\dec\vp8.c(588) : warning C4701: potentially 
uninitialized local variable 'nz_dc' used
third_party\libwebp\files\dec\vp8.c(589) : warning C4701: potentially 
uninitialized local variable 'nz_ac' used

third_party\libwebp\files\dec\buffer.c(77) : warning C4244: '=' : conversion 
from 'uint64_t' to 'int', possible loss of data
third_party\libwebp\files\dec\buffer.c(99) : warning C4244: '=' : conversion 
from 'uint64_t' to 'int', possible loss of data
third_party\libwebp\files\dec\buffer.c(109) : warning C4244: '=' : conversion 
from 'uint64_t' to 'int', possible loss of data
third_party\libwebp\files\dec\buffer.c(115) : warning C4244: '=' : conversion 
from 'uint64_t' to 'int', possible loss of data

third_party\libwebp\files\dec\io.c(259) : warning C4244: '=' : conversion from 
'int64_t' to 'int', possible loss of data
third_party\libwebp\files\dec\io.c(285) : warning C4244: 'initializing' : 
conversion from 'int64_t' to 'const int', possible loss of data

webp_src + 'dec/buffer.c',  # warning 4244
webp_src + 'dec/io.c',  # warning 4244
webp_src + 'dec/vp8.c',  # warning 4201

Original issue reported on code.google.com by fbarch...@google.com on 14 Mar 2012 at 5:31

GoogleCodeExporter commented 9 years ago
Yes there are few remaining in HEAD and they differ between x86/x64. Your 
target + warning level?

Original comment by jz...@google.com on 14 Mar 2012 at 8:37

GoogleCodeExporter commented 9 years ago
Seems I missed the obvious in the error text, though I see a different a 
somewhat different set in my 2008 x86/x64 builds. These are still valid though.

https://gerrit.chromium.org/gerrit/#change,18418

Original comment by jz...@google.com on 17 Mar 2012 at 3:21

GoogleCodeExporter commented 9 years ago
Warnings should be silent now, some related changes:

a6cdf71 muxedit: a few more size_t changes
11ae46a alpha.c: quiet some size_t -> int conversion warnings
d96e722 huffman: quiet int64 -> int conversion warning
532020f lossless: remove some size_t -> int conversions
95667b8 mux: use size_t consistently
80256b8 enc_sse2 add missing stdlib.h include
9b3d1f3 decode.h: use size_t consistently
1c92bd3 vp8io: use size_t for buffer size
90ead71 fix some more uint32_t -> size_t typing
cbe705c webp/decode.h: fix prototypes
6860c2e fix some uint32_t -> size_t typing

Original comment by jz...@google.com on 4 May 2012 at 1:43