[FFmpeg-devel] [RFC] comment mpegvideo_split

Michael Niedermayer michaelni
Thu Apr 3 15:48:10 CEST 2008


On Thu, Apr 03, 2008 at 03:08:32PM +0200, Baptiste Coudurier wrote:
> Hi,
> 
> Michael Niedermayer wrote:
> > On Wed, Mar 19, 2008 at 04:11:37PM +0100, Baptiste Coudurier wrote:
> >> Baptiste Coudurier wrote:
> >>> Michael Niedermayer wrote:
> >>>> On Tue, Mar 18, 2008 at 01:16:34AM +0100, Baptiste Coudurier wrote:
> >>>>> Hi,
> >>>>>
> >>>>> Currently, mpegvideo split function causes problems with containers
> >>>>> setting AVFMT_GLOBAL_HEADER, mov and mp4 in particular, because
> >>>>> MPEG-1/2 in those containers are to be muxed normally.
> >>>>>
> >>>>> I don't know any particular usage of the mpegvideo split function, so I
> >>>>> tend to want to comment it, to finally fix this issue.
> >>>>>
> >>>>> Anyone see another better alternative ?
> >>>> After a few more seconds :)
> >>>> The muxers could just dump extradata for the case of mpeg1/2 before keyframes,
> >>>> that should fix it i think. Its of course a little odd to first extract and
> >>>> then put it back in the muxer.
> >>>>
> >>> Would the attached patch be ok ?
> >>>
> >> This one is more correct, messy though.
> >>
> >> [...]
> >>  
> >> +    if (dump_extradata) {
> >> +        put_buffer(pb, trk->vosData, trk->vosLen);
> >> +        size -= trk->vosLen;
> >> +    }
> >>      put_buffer(pb, pkt->data, size);
> >>  
> >>      put_flush_packet(pb);
> > 
> > ok, its not beautifull but i dont know a better solution. Maybe some
> > auto insertion of a bitstream filter, that could also be usefull to
> > simplify other cases ...> > Anyway i think its better to commit this, we can always revert it if
> > someone does find a better solution.
> > 
> 
> I was thinking about this other solution, and it seems to work, though
> It acts a bit behind the user's back.

I wonder if this will work with -vcodec copy from nut.

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

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- 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/20080403/b615b16f/attachment.pgp>



More information about the ffmpeg-devel mailing list