[FFmpeg-devel] [PATCH] Implement guessed_pts in avcodec_decode_video2.

Jean-Daniel Dupas devlists
Mon Jan 31 14:54:55 CET 2011


Le 31 janv. 2011 ? 13:37, M?ns Rullg?rd a ?crit :

> Jean-Daniel Dupas <devlists at shadowlab.org> writes:
> 
>> Le 30 janv. 2011 ? 20:39, M?ns Rullg?rd a ?crit :
>> 
>>> Nicolas George <nicolas.george at normalesup.org> writes:
>>> 
>>>> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
>>>> ---
>>>> cmdutils.c           |   27 ---------------------------
>>>> cmdutils.h           |   24 ------------------------
>>>> ffmpeg.c             |   14 ++++++--------
>>>> ffplay.c             |   10 +++-------
>>>> libavcodec/avcodec.h |   20 ++++++++++++++++++++
>>>> libavcodec/utils.c   |   43 ++++++++++++++++++++++++++++++++++++++++++-
>>>> 6 files changed, 71 insertions(+), 67 deletions(-)
>>>> 
>>>> This patch moves the guess_correct_pts function from cmdutils to libavcodec.
>>>> The feature is available through a new field in AVFrame, guessed_pts.
>>>> 
>>>> make test and ffplay still work, but I would like to have some more time to
>>>> read it carefully, but I have things to do in the next few days, and this
>>>> feature was discussed in another thread, so here the current version. It
>>>> will conflict with some patches that will certainly soon be applied, so I
>>>> will update and submit it again anyway.
>>> 
>>> What is all this guessing supposed to achieve anyway?  It makes me
>>> feel uneasy
>> 
>> Try to play H.264 with reordered frame in avi container without this
>> guess, and you will see why it is necessary.
> 
> AVI does not have timestamps (only an implied DTS), so the PTS would
> be calculated by counting decoded frames and multiplying by the frame
> rate.


So why the demuxer is providing wrong information ? 


-- Jean-Daniel







More information about the ffmpeg-devel mailing list