Re: [FFmpeg-devel] [PATCH] lavc/utils.c: Make sure skip_samples never goes negative.

2016-11-10 Thread Michael Niedermayer
On Thu, Nov 10, 2016 at 01:30:34PM +, Derek Buitenhuis wrote: > On 11/4/2016 10:31 PM, Sasi Inguva wrote: > > -avctx->internal->skip_samples -= frame->nb_samples; > > +avctx->internal->skip_samples = FFMAX(0, > > avctx->internal->skip_samples - frame->nb_samples); > >

Re: [FFmpeg-devel] [PATCH] lavc/utils.c: Make sure skip_samples never goes negative.

2016-11-10 Thread Derek Buitenhuis
On 11/4/2016 10:31 PM, Sasi Inguva wrote: > -avctx->internal->skip_samples -= frame->nb_samples; > +avctx->internal->skip_samples = FFMAX(0, > avctx->internal->skip_samples - frame->nb_samples); LGTM. - Derek ___ ffmpeg-devel ma

Re: [FFmpeg-devel] [PATCH] lavc/utils.c: Make sure skip_samples never goes negative.

2016-11-09 Thread Sasi Inguva
ping. On Sat, Nov 5, 2016 at 4:01 AM, Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavcodec/utils.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index 87de15f..d6dca18 100644 > --- a/libavcodec/utils.c > +++ b/

[FFmpeg-devel] [PATCH] lavc/utils.c: Make sure skip_samples never goes negative.

2016-11-04 Thread Sasi Inguva
Signed-off-by: Sasi Inguva --- libavcodec/utils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 87de15f..d6dca18 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2391,7 +2391,7 @@ int attribute_align_arg avcodec_deco

Re: [FFmpeg-devel] [PATCH] lavc/utils.c: Make sure skip_samples never goes negative.

2016-11-04 Thread Sasi Inguva
yeah. that is better. On Thu, Nov 3, 2016 at 3:41 PM, Hendrik Leppkes wrote: > On Thu, Nov 3, 2016 at 11:31 PM, Sasi Inguva > wrote: > > Signed-off-by: Sasi Inguva > > --- > > libavcodec/utils.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/libavcodec/utils

Re: [FFmpeg-devel] [PATCH] lavc/utils.c: Make sure skip_samples never goes negative.

2016-11-03 Thread Hendrik Leppkes
On Thu, Nov 3, 2016 at 11:31 PM, Sasi Inguva wrote: > Signed-off-by: Sasi Inguva > --- > libavcodec/utils.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/utils.c b/libavcodec/utils.c > index 87de15f..a33afaa 100644 > --- a/libavcodec/utils.c > +++ b/libav

[FFmpeg-devel] [PATCH] lavc/utils.c: Make sure skip_samples never goes negative.

2016-11-03 Thread Sasi Inguva
Signed-off-by: Sasi Inguva --- libavcodec/utils.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 87de15f..a33afaa 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2391,7 +2391,9 @@ int attribute_align_arg avcodec_d