[FFmpeg-devel] [PATCH] movie video source

[Stefano Sabatini]

On date Tuesday 2011-01-04 23:48:24 +0100, Stefano Sabatini encoded:
> On date Tuesday 2011-01-04 18:48:48 +0100, Nicolas George encoded:
> > Le quintidi 15 niv?se, an CCXIX, Stefano Sabatini a ?crit?:
> > > What do you suggest to do, to make the cmdutils.c PtsCorrectionContext
> > > API public and use it (av_pts_correction_init(),
> > > av_pts_correction_guess()) or to duplicate the code?
> > 
> > Something like that was evoked three weeks ago in the thread about the API
> > example for lavfi. Since then, I thought I might try to implement it.
> > 
> > My idea was to add the PtsCorrectionContext structure to the end of
> > AVCodecContext and add some sort of avcodec_set_stream_time_base function:
> > this way, avcodec_decode_video2 would be able to use the pts and dts fields
> > in AVPacket directly without depending on libavformat.
> 
> My idea was to implement a dumb transposition of the correct_pts API
> in cmdutils.c:
> 
> AVPTSCorrectionContext
> void av_pts_correction_init(AVPTSCorrectionContext *ctx);
> int64_t av_pts_correction_guess(AVPTSCorrectionContext *ctx, int64_t pts, int64_t dts);
> 
> this would be defined in libavcodec/avcodec.h (as I don't see any
> dependency on libavformat).

Michael? (I'm not going to even start coding if the design will be
rejected).

> I don't know if is a good idea to add the
> AVPTSCorrectionContext to AVCodecContext, I suppose the idea would be
> to move all the TS reordering code in ffplay.c to
> avcodec_decode_video2() (and add another parameter decoder_reorder_pts
> to AVCodecContext), anyway I'm not the best person to judge if this is
> acceptable.
-- 
FFmpeg = Fierce and Formidable Mythic Puristic Erratic God