On Mon, 19 Sep 2016 21:20:08 +0200
Carl Eugen Hoyos wrote:
> 2016-09-19 20:32 GMT+02:00 wm4 :
>
> > But I guess you don't care, neither did whoever pushed this
> > low quality patch before the patch was fixed?
>
> I believe you didn't oppose the current mailing list rules so
> please refrain
2016-09-19 20:32 GMT+02:00 wm4 :
> But I guess you don't care, neither did whoever pushed this
> low quality patch before the patch was fixed?
I believe you didn't oppose the current mailing list rules so
please refrain from personal attacks.
Thank you, Carl Eugen
___
On Tue, 9 Aug 2016 18:48:21 -0700
Sasi Inguva wrote:
> Signed-off-by: Sasi Inguva
> ---
> libavcodec/avcodec.h | 6 ++
> libavcodec/utils.c | 14 +-
> libavcodec/version.h | 2 +-
> 3 files changed, 20 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/avcodec.h b/l
skip_samples shouldn't be negative. But it's not enforced because it is an
int field. Changed the condition to equality. Added an assert where
skip_samples is populated.
On Sun, Aug 14, 2016 at 5:40 AM, Clément Bœsch wrote:
> On Tue, Aug 09, 2016 at 06:48:21PM -0700, Sasi Inguva wrote:
> [...]
>
Signed-off-by: Sasi Inguva
---
libavcodec/avcodec.h | 6 ++
libavcodec/utils.c | 15 ++-
libavcodec/version.h | 2 +-
3 files changed, 21 insertions(+), 2 deletions(-)
diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
index 3b21537..d68da01 100644
--- a/libavcodec/avco
On Tue, Aug 09, 2016 at 06:48:21PM -0700, Sasi Inguva wrote:
[...]
> +if ((frame->flags & AV_FRAME_FLAG_DISCARD) &&
> avctx->internal->skip_samples <= 0) {
skip_samples can be negative?
if so, that looks like a behaviour change we want documented
if not, you probably want to check again
Signed-off-by: Sasi Inguva
---
libavcodec/avcodec.h | 6 ++
libavcodec/utils.c | 14 +-
libavcodec/version.h | 2 +-
3 files changed, 20 insertions(+), 2 deletions(-)
diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
index 3b21537..d68da01 100644
--- a/libavcodec/avcod