Re: various warnings in 5.1.4 and 5.2.0-work4 with icc/msvc64/splint

[David Manura <dm.lua@...>]

On Tue, Oct 26, 2010 at 8:01 AM, Roberto Ierusalimschy
<roberto@inf.puc-rio.br> wrote:
>> http://msdn.microsoft.com/en-us/library/ke55d167(v=VS.100).aspx ....
> They really could not care less for portability...

I guess you mean the i64 thing [1], with the only recent addition of
C99 stdint.h (but not inttypes.h) [2-3].

[1] http://stackoverflow.com/questions/1264059/in-c-what-is-the-difference-between-1-and-1i64
[2] http://connect.microsoft.com/VisualStudio/feedback/details/99131/c99-header-stdint-h-missing
[3] http://connect.microsoft.com/VisualStudio/feedback/details/375930/tr1-support-was-c99-support

>> The below patch makes it happy. [...]
> It does not seem to cover all shift-warnings in your original message...

Concerning the 5.2 warnings, most seem to be about things like
implicitly, rather than explicitly, casting a wider type (size_t or
ptrdiff_t) into a narrower type (int), with the possible data loss
that might be unintended.  gcc (64 not 32) gives similar warnings and
others if you use -Wconversion (not just -Wall).