[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