Re: [FFmpeg-devel] [PATCH] libavfilter: constify filter list

2018-02-01 Thread Brian Matherly
On 2/1/2018 4:03 AM, Nicolas George wrote: Regarding the efficiency of avfilter_next(), I think we should not care. Almost nobody uses that function outside lavfi anyway. If it proves a concern for some application, we can deal with it when it is found. For what it's worth, I use avfilter_next(

Re: [FFmpeg-devel] [PATCH v3] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-27 Thread Brian Matherly
On 9/26/2017 8:38 PM, Michael Niedermayer wrote: On Sun, Sep 24, 2017 at 10:13:11PM -0500, Brian Matherly wrote: From: Brian Matherly Correctly set the interlaced_frame and top_field_first fields when pic_struct indicates paired fields. --- libavcodec/hevc_sei.c | 4

[FFmpeg-devel] [PATCH v4] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-27 Thread Brian Matherly
From: Brian Matherly Correctly set the interlaced_frame and top_field_first fields when pic_struct indicates paired fields. --- libavcodec/hevc_sei.c | 4 ++-- tests/fate/hevc.mak | 6 +- tests/ref/fate/hevc-paired-fields | 16 3 files changed

[FFmpeg-devel] [PATCH v3] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-24 Thread Brian Matherly
From: Brian Matherly Correctly set the interlaced_frame and top_field_first fields when pic_struct indicates paired fields. --- libavcodec/hevc_sei.c | 4 +- tests/fate/hevc.mak | 6 +- tests/ref/fate/hevc-paired-fields | 120

Re: [FFmpeg-devel] [PATCH] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-24 Thread Brian Matherly
On 9/23/2017 5:22 PM, Michael Niedermayer wrote: On Thu, Sep 14, 2017 at 08:39:19PM -0500, Brian Matherly wrote: Correctly set frame.interlaced and frame.top_field_first when pic_struct indicates paired fields. Do you have a (small) sample that gets fixed by this ? Can you make a fate test

[FFmpeg-devel] [PATCH v2] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-24 Thread Brian Matherly
From: Brian Matherly Correctly set the interlaced_frame and top_field_first fields when pic_struct indicates paired fields. --- libavcodec/hevc_sei.c | 4 +- tests/fate/hevc.mak | 3 + tests/ref/fate/hevc-paired-fields | 120

[FFmpeg-devel] [PATCH v2] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-24 Thread Brian Matherly
From: Brian Matherly Correctly set the interlaced_frame and top_field_first fields when pic_struct indicates paired fields. --- libavcodec/hevc_sei.c | 4 +- tests/fate/hevc.mak | 3 + tests/ref/fate/hevc-paired-fields | 120

Re: [FFmpeg-devel] [PATCH] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-21 Thread Brian Matherly
On 9/14/2017 8:39 PM, Brian Matherly wrote: Correctly set frame.interlaced and frame.top_field_first when pic_struct indicates paired fields. --- libavcodec/hevc_sei.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/hevc_sei.c b/libavcodec/hevc_sei.c index

[FFmpeg-devel] [PATCH] avcodec/hevc_sei: Support HEVC paired fields.

2017-09-14 Thread Brian Matherly
Correctly set frame.interlaced and frame.top_field_first when pic_struct indicates paired fields. --- libavcodec/hevc_sei.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/hevc_sei.c b/libavcodec/hevc_sei.c index cd55d50..d0f9966 100644 --- a/libavcodec/hevc_sei.

Re: [FFmpeg-devel] [PATCH] avcodec/scpr: optimize shift loop.

2017-09-09 Thread Brian Matherly
On 9/9/2017 1:27 PM, Michael Niedermayer wrote: +// If the image is sufficiently aligned, compute 8 samples at once +if (!(((uintptr_t)dst) & 7)) { +uint64_t *dst64 = (uint64_t *)dst; +int w = avctx->width>>1; +for (x = 0; x

Re: [FFmpeg-devel] [PATCH] Add A53 Closed Captions to MPEG header if they are available.

2017-06-15 Thread Brian Matherly
Indeed, multiple entries of the same type are really a bad idea and we basically made them impossible with stream sidedata, although maybe not with frame side data yet. We should not add API for them or encourage their use. If there is a real need for multiple of the same type, maybe the type sh

Re: [FFmpeg-devel] [PATCH] Add A53 Closed Captions to MPEG header if they are available.

2017-06-06 Thread Brian Matherly
Indeed, multiple entries of the same type are really a bad idea and we basically made them impossible with stream sidedata, although maybe not with frame side data yet. We should not add API for them or encourage their use. If there is a real need for multiple of the same type, maybe the type sh

Re: [FFmpeg-devel] [PATCH] Add "split" mode to tinterlace filter.

2015-03-28 Thread Brian Matherly
>> This mode is the opposite of the "merge" mode. >> --- >> This patch adds a new mode to tinterlace which performs the opposite operation >> as the "merge" mode. >> >> My primary motivation is that I have been working with Derek Buitenhuis to see >> about adding interlace support to the libx265 en

[FFmpeg-devel] [PATCH] Add "split" mode to tinterlace filter.

2015-03-28 Thread Brian Matherly
From: Brian Matherly This mode is the opposite of the "merge" mode. --- This patch adds a new mode to tinterlace which performs the opposite operation as the "merge" mode. My primary motivation is that I have been working with Derek Buitenhuis to see about adding inte