[FFmpeg-devel] [PATCH] ARMovie/RPL demuxer rev4

Reimar Döffinger Reimar.Doeffinger
Sat Mar 29 10:26:55 CET 2008


On Sat, Mar 29, 2008 at 02:47:56AM +0100, Michael Niedermayer wrote:
> On Sat, Mar 29, 2008 at 02:31:29AM +0100, Reimar D?ffinger wrote:
> > Which IMO is a really horrible idea, since that means the demuxer will
> > behave differently on 32 and 64 bit systems.
> > You could use one of the SCN?64 defines and uint64_t...
> 
> Yes, iam glad you found that issue in time, after that is fixed we will
> finally be able to play all thouse >4gb ARMovie/RPL files which we have
> been collecting. ;)

Well, I would be equally fine with just e.g. %u and int, but code that
behaves one way on one half of the architectures and another way on the
other have is just a pain to debug, which I consider it evil even if it
is of little consequence in this case probably.

Greetings,
Reimar D?ffinger




More information about the ffmpeg-devel mailing list