[FFmpeg-devel] [PATCH] RoQ muxer

Michael Niedermayer michaelni
Mon May 7 16:19:54 CEST 2007


Hi

On Mon, May 07, 2007 at 12:26:36AM +0200, Vitor wrote:
> Michael Niedermayer wrote:
> >you cannot use av_dup_packet() on av_mallocz() data (theres no packet to
> >dup(licate) )
> >  
> Oops... Bad copy pasting...
> 
> >this is fairly complex considering it seems to do just something like:
> >  
> I could not make it as simple as you suggested, but it is already pretty 
> better.
> 
> -Vitor

[...]

> +    int ret = av_interleave_packet_per_dts(s, pkt, in, flush);

why the "double" interleave

[...]
> +    /* If there are 2 frames of video pending and no audio, assume there's no
> +     * more audio */
> +    if (!ctx->queueBase[0] && ctx->queueBase[1] && ctx->queueBase[1]->next)
> +        ctx->nxtPkt = PACKETTYPE_VIDEO;

what is this code good for / in which case would it fail if this wherent here?

[...}
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I count him braver who overcomes his desires than him who conquers his
enemies for the hardest victory is over self. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070507/dcf8e2a9/attachment.pgp>



More information about the ffmpeg-devel mailing list