[FFmpeg-devel] [FFmpeg-cvslog] lavc: allow subtitle text format to be ASS without timing

[Clément Bœsch]

On Fri, Mar 04, 2016 at 04:22:44PM +0100, Michael Niedermayer wrote:
> On Fri, Feb 26, 2016 at 10:01:51PM +0100, Clément Bœsch wrote:
> > ffmpeg | branch: master | Clément Bœsch <u at pkh.me> | Wed Jan  6 13:43:23 2016 +0100| [29412821241050c846dbceaad4b9752857659977] | committer: Clément Bœsch
> [...]
> > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> > index 7a56899..d3e035a 100644
> > --- a/libavcodec/avcodec.h
> > +++ b/libavcodec/avcodec.h
> > @@ -3285,6 +3285,10 @@ typedef struct AVCodecContext {
> >  #define FF_SUB_CHARENC_MODE_AUTOMATIC    0  ///< libavcodec will select the mode itself
> >  #define FF_SUB_CHARENC_MODE_PRE_DECODER  1  ///< the AVPacket data needs to be recoded to UTF-8 before being fed to the decoder, requires iconv
> >  
> > +    int sub_text_format;
> > +#define FF_SUB_TEXT_FMT_ASS              0
> > +#define FF_SUB_TEXT_FMT_ASS_WITH_TIMINGS 1
> > +
> >      /**
> >       * Skip processing alpha if supported by codec.
> >       * Note that if the format uses pre-multiplied alpha (common with VP6,
> 
> doesnt this break ABI ?
> 

Ah, mmmh... We're not that far away from the bottom, aren't all the
following fields supposed to be accessed through AVoption or other
accessors?

How to proceed here, should I just move it at the bottom of the struct
since it was introduced recently?

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160304/a0365edf/attachment.sig>