[MPlayer-dev-eng] serious warnings and bugs

[Arpi]

Hi,

> > For instance in libmpdemux/cache2.c there is no #ifdef, althoug off_t
> > may be 8B (so it should be %llX) :
> 
> Right - I remember, I think, taht A'rpi didn't like the patch with all
> those changes in it (there were quite a few).  Given that most are debug
> messages, and that for you see the low 32bits as it is - which is usually
> fine, I didn't push it.

I said that typecasting should do (int) instead of (long long),
as the long long (and %llX) has portability problems.
at least, on AIX and linux printf uses different format string for 64bit
integers. (i know it from advanced mc project)


A'rpi / Astral & ESP-team

--
mailto:arpi at thot.banki.hu
http://esp-team.scene.hu