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

Michael Niedermayer michaelni
Tue Oct 12 20:18:25 CEST 2010


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


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

It is not what we do, but why we do it that matters.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101012/1bc66a01/attachment.pgp>



More information about the ffmpeg-devel mailing list