[FFmpeg-devel] [PATCH] yuv4mpeg: support concatenated files

Michael Niedermayer michaelni
Thu Apr 1 19:34:02 CEST 2010


On Sat, Mar 27, 2010 at 10:09:01PM +0100, Reimar D?ffinger wrote:
> On Sat, Mar 27, 2010 at 09:55:24PM +0100, Michael Niedermayer wrote:
> > On Sat, Mar 27, 2010 at 05:35:55PM +0100, Reimar D?ffinger wrote:
> > > Hello,
> > > attached patch would allow for the yuv4mpeg demuxer to support concatenated
> > > files.
> > > I guess strictly those files are invalid, but it is the only somewhat sane way
> > > I know to convert MPEG-files with size or aspect changes to uncompressed.
> > > Even after this patch, seeking in such files won't work right though.
> > 
> > >  yuv4mpeg.c |   69 ++++++++++++++++++++++++++++++++++++++-----------------------
> > >  1 file changed, 43 insertions(+), 26 deletions(-)
> > > b052f43302629f6b189fbdf642007108db0df47c  yu4mconcat.diff
> > > Index: libavformat/yuv4mpeg.c
> > > ===================================================================
> > > --- libavformat/yuv4mpeg.c	(revision 22697)
> > > +++ libavformat/yuv4mpeg.c	(working copy)
> > > @@ -27,6 +27,7 @@
> > >  struct frame_attributes {
> > >      int interlaced_frame;
> > >      int top_field_first;
> > > +    int stream_id;
> > >  };
> > >  
> > >  #if CONFIG_YUV4MPEGPIPE_MUXER
> > 
> > > @@ -184,22 +185,14 @@
> > >  #endif
> > >  
> > >  /* Header size increased to allow room for optional flags */
> > > -#define MAX_YUV4_HEADER 80
> > > +#define MAX_HEADER 80
> > 
> > cosmetic ?
> 
> Well, it was a merging of two different defines, that's why it's a new name.
> But I guess the MAX_YUV4_HEADER name works just as well, not sure if the
> resulting patch is actually better or worse to read.

shouldnt the starttime of each part be set to the start+duration of the
previous ?

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

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- 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/20100401/0be304fd/attachment.pgp>



More information about the ffmpeg-devel mailing list