[FFmpeg-devel] [PATCH] matroskadec, matroskadec, srtenc: Read/Write duration for subtitles. (v2)

[Nicolas George]

Le nonidi 19 thermidor, an CCXX, Philip Langdale a écrit :
> Ugh. This is such a mess. If we really can't move mkv to use duration,
> then we've got to go in the other direction.
> 
> Maybe:
> 
> #define SUBTITLE_DURATION convergence_duration
> 
> and use pkg->SUBTITLE_DURATION for now and switch the
> define when we can finally break compatibility.

I would rather suggest:

/**
 * Please use this function instead of accessing AVFrame.duration directly
 */
int64_t avframe_get_subtitle_duration(const AVFrame *avframe);

And then we have all liberty to adjust things.

Regards,

-- 
  Nicolas George
-------------- 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/20120805/9b7dd6f7/attachment.asc>