[MPlayer-dev-eng] [PATCH] Yet another batch of warning fixes :->

Dominik Mierzejewski dominik at rangers.eu.org
Tue Dec 10 14:16:34 CET 2002


On Tuesday, 10 December 2002, Arpi wrote:
> Hi,
> 
> > > 
> > > it'll break code on 64bit platforms badly
> > 
> > Wait a minute... will our win32 loader even *work* on 64bit platforms?
> 
> of course not.
> 
> but the .h files in loader/wine/* are used by demuxers (avi, asf and other
> m$ formats) so they must be correct even on 64bit archs (otherwise the
> compiled struct size will differ from the one stored in the file's header)
> 
> avifile "solved" this issue by duplicating all those structures in its
> include/ dir (using int32_t etc instead of DWORD) and their demuxers
> don't use files from loader/wine/* at all.

Of course! I knew I was missing something. Thank you so much for being
so patient with me. So, how about typedef'ing DWORD etc. as int32_t etc.
in our headers?
 
-- 
MPlayer RPMs maintainer: http://www.piorunek.pl/~dominik/linux/pkgs/mplayer/
"The Universe doesn't give you any points for doing things that are easy."
        -- Sheridan to Garibaldi in Babylon 5:"The Geometry of Shadows"



More information about the MPlayer-dev-eng mailing list