[FFmpeg-devel] [PATCH] add avcodec_get_subtitle_defaults() to initialize AVSubtitle struct
Michael Niedermayer
michaelni at gmx.at
Wed Apr 6 02:31:38 CEST 2011
On Wed, Apr 06, 2011 at 12:30:23AM +0200, Aurelien Jacobs wrote:
> On Tue, Apr 05, 2011 at 06:26:22PM +0200, Reimar Döffinger wrote:
> > On Tue, Apr 05, 2011 at 10:04:33AM +0200, Aurelien Jacobs wrote:
> > > Call this new function before decode() to replace the custom and
> > > inconsistant initialization in various decoders.
> > > This function is equivalent to avcodec_get_frame_defaults() for AVFrame.
> >
> > Any real reason to make it public?
>
> Not really. I just did it the same way as avcodec_get_frame_defaults().
>
> > We don't have much experience with subtitles yet, so I'd prefer if
> > it wasn't public - who knows what kind of issues we might detect
> > with it.
>
> Agree. We can make it public when someone need it.
> New patch attached with avcodec_get_subtitle_defaults() as a static
> function.
>
> Aurel
> ass.c | 5 -----
> ass.h | 7 -------
> assdec.c | 2 --
> dvbsubdec.c | 3 ---
> dvdsubdec.c | 1 -
> pgssubdec.c | 1 -
> srtdec.c | 2 --
> utils.c | 7 +++++++
> xsubdec.c | 2 --
> 9 files changed, 7 insertions(+), 23 deletions(-)
> 7585cd15b07fc9b0db143814855b3e684a7dd910 0001-add-avcodec_get_subtitle_defaults-to-initialize-AVSu.patch
> From 383f5868f65424b4c08380a18a2e1b6b4ac1b33d Mon Sep 17 00:00:00 2001
> From: Aurelien Jacobs <aurel at gnuage.org>
> Date: Tue, 5 Apr 2011 09:04:48 +0200
> Subject: [PATCH] add avcodec_get_subtitle_defaults() to initialize AVSubtitle struct
LGTM
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110406/897e4606/attachment.asc>
More information about the ffmpeg-devel
mailing list