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

Reimar Döffinger Reimar.Doeffinger
Sat Mar 27 22:09:01 CET 2010


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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: yu4mconcat.diff
Type: text/x-diff
Size: 3908 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100327/0494a0df/attachment.diff>



More information about the ffmpeg-devel mailing list