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

[Aurelien Jacobs]

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.

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: subtitle_codec_parameters.diff
Type: text/x-diff
Size: 453 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100810/d5c03556/attachment.diff>