[FFmpeg-devel] [PATCH] Move allocation of AVOutputStream to an earlier place

[Stefano Sabatini]

On date Tuesday 2010-10-12 20:18:25 +0200, Michael Niedermayer encoded:
> On Tue, Oct 12, 2010 at 05:04:48PM +0200, Nicolas George wrote:
> > Hi.
> > 
> > The first attached patch moves the allocation of the AVOutputStream
> > structure earlier in the code flow, in the new_video_stream /
> > new_audio_stream / new_subtitle_stream functions.
> > 
> > The second patch uses it to place the bitstream_filters variable in the
> > AVOutputStream, thus greatly simplifying its handling and reducing code
> > duplication.
> > 
> > make test still succeeds, but it does not seem to cover bitstream filters
> > anyway; a simple test with -absf noise succeeds too.
> > 
> > Regards,
> > 
> > -- 
> >   Nicolas George
> 
> >  ffmpeg.c |   42 ++++++++++++++++++++++++++++++++----------
> >  1 file changed, 32 insertions(+), 10 deletions(-)
> > 1c28303ad8c2b52dd2a5c9f2153599687143299d  ffmpeg-outputstream-20101012-1650-01-allocearly.diff
> >  ffmpeg.c |   42 ++++++++++++++++++++++++++++++++----------
> >  1 files changed, 32 insertions(+), 10 deletions(-)
> 
> ok
> 
> 
> [...]
> >  ffmpeg.c |   36 +++++++++++-------------------------
> >  1 file changed, 11 insertions(+), 25 deletions(-)
> > faccccfbb4927bb2e2af6a8c77ca7db0e115a2fa  ffmpeg-outputstream-20101012-1650-02-bsfs.diff
> >  ffmpeg.c |   36 +++++++++++-------------------------
> >  1 files changed, 11 insertions(+), 25 deletions(-)
> 
> ok

Both applied.
-- 
FFmpeg = Faithless & Furious Majestic Proud Exxagerate Genius