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

Eli Friedman eli.friedman
Fri Mar 28 21:09:18 CET 2008


On Fri, Mar 28, 2008 at 6:42 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
>  > +            // There should always be frames_per_chunk frames in a chunk.
>  > +            pkt->duration = rpl->frames_per_chunk;
>
>  There should be only 1 frame in each AVPacket

There's no possible way to enforce that correctly for an unknown
codec; in general, the frame boundaries can be anywhere within the
chunk.   Like the comment in the patch says, Escape 124 gets
special-cased, Escape 122 always has rpl->frames_per_chunk == 1, and I
haven't coded the special-casing for Escape 130. (Taking a look at the
description, I think the special-casing for Escape 130 is the same as
for Escape 124, but I can't really test without a working Escape 130
implementation.).

What exactly am I supposed to do here?

-Eli




More information about the ffmpeg-devel mailing list