[FFmpeg-devel] [PATCH 2/3] ffmpeg: open encoders after decoders

Michael Niedermayer michaelni
Wed Aug 11 18:22:22 CEST 2010


On Tue, Aug 10, 2010 at 10:39:37PM +0200, Aurelien Jacobs wrote:
> On Tue, Aug 10, 2010 at 02:10:35PM +0200, Michael Niedermayer wrote:
> > On Sun, Aug 08, 2010 at 02:26:57AM +0200, Aur?lien Jacobs wrote:
> > > From: Aurelien Jacobs <aurel at gnuage.org>
> > > 
> > > ---
> > >  ffmpeg.c |   46 +++++++++++++++++++++++-----------------------
> > >  1 files changed, 23 insertions(+), 23 deletions(-)
> > 
> > this should be unneeded, av_find_stream_info() should cause all the things
> > to be set
> 
> It currently don't. So I've tried to force av_find_stream_info() to fill
> the subtitle_header field with the attached patch. With this,
> av_find_stream_info() indeed fills subtitle_header, but after it finishes
> probing parameters, it calls avcodec_close() which effectively free the
> subtitle_header buffer.
> So after av_find_stream_info() call, subtitle_header is NULL again...
> 
> Any better alternative ?


> To me it seems somewhat logical to initialize decoders before encoders.

that might require changing applications and might be non trivial for some.

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

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 190 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100811/3120d118/attachment.pgp>



More information about the ffmpeg-devel mailing list