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

Michael Niedermayer michaelni
Sat Mar 29 13:14:15 CET 2008


On Sat, Mar 29, 2008 at 10:26:55AM +0100, Reimar D?ffinger wrote:
> 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.

It should be uint64_t of course, i didnt say anything else did i?

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is not what we do, but why we do it that matters.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080329/563496ef/attachment.pgp>



More information about the ffmpeg-devel mailing list