Re: [FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-05 Thread wm4
On Tue, 4 Oct 2016 20:26:49 +0200 Michael Niedermayer wrote: > On Tue, Oct 04, 2016 at 06:33:48PM +0200, wm4 wrote: > > Some API users (in particular ffmpeg.c) check the best effort timestamp > > only. > > --- > > Using guess_correct_pts() - not sure what I was thinking.

Re: [FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-04 Thread Michael Niedermayer
On Tue, Oct 04, 2016 at 06:33:48PM +0200, wm4 wrote: > Some API users (in particular ffmpeg.c) check the best effort timestamp > only. > --- > Using guess_correct_pts() - not sure what I was thinking. > --- > libavcodec/utils.c | 9 - > 1 file changed, 8 insertions(+), 1 deletion(-) LGTM

[FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-04 Thread wm4
Some API users (in particular ffmpeg.c) check the best effort timestamp only. --- Using guess_correct_pts() - not sure what I was thinking. --- libavcodec/utils.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index

Re: [FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-04 Thread Michael Niedermayer
On Sun, Oct 02, 2016 at 06:56:48PM +0200, wm4 wrote: > Some API users (in particular ffmpeg.c) check the best effort timestamp > only. > --- > Still undecided if this is the right approach. > --- > libavcodec/utils.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git

Re: [FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-03 Thread Timo Rothenpieler
On 10/2/2016 6:56 PM, wm4 wrote: > Some API users (in particular ffmpeg.c) check the best effort timestamp > only. > --- > Still undecided if this is the right approach. > --- > libavcodec/utils.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/utils.c

[FFmpeg-devel] [PATCH] lavc: set best effort timestamp if unset when using new decode API

2016-10-02 Thread wm4
Some API users (in particular ffmpeg.c) check the best effort timestamp only. --- Still undecided if this is the right approach. --- libavcodec/utils.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index ef3da65..1875a69 100644