[FFmpeg-devel] FW: RFQ on patch for decklink warnings

bump From: ffmpeg-devel on behalf of "ffmpeg-devel-boun...@ffmpeg.org" Reply-To: FFmpeg development discussions and patches Date: Tuesday, May 23, 2017 at 4:06 PM To: FFmpeg development discussions and

[FFmpeg-devel] RFQ on patch for decklink warnings

Afternoon all, I’m looking at fixing some warnings on deprecated api calls in the decklink drivers. I’ve got a patch ready, but it looks like I might be leaking memory somewhere. In the old code using the av_dup_packet() is a free() of some kind required in code that I might be able to look

Re: [FFmpeg-devel] [PATCH] - libavdevices/decklink_*.cpp: formatting change to remove uneeded spaces in initializers

/decklink_*.cpp: formatting change to remove uneeded spaces in initializers On Tue, Aug 2, 2016 at 10:17 PM Felt, Patrick <patrick.f...@echostar.com<mailto:patrick.f...@echostar.com>> wrote: --- libavdevice/decklink_common.cpp | 10 +- libavdevice/decklink_dec.cpp| 18

[FFmpeg-devel] [PATCH] - libavdevices/decklink_*.cpp: formatting change to remove uneeded spaces in initializers

--- libavdevice/decklink_common.cpp | 10 +- libavdevice/decklink_dec.cpp| 18 +- libavdevice/decklink_enc.cpp| 26 +- 3 files changed, 27 insertions(+), 27 deletions(-) cosmetic2.diff Description: cosmetic2.diff

Re: [FFmpeg-devel] [PATCH] - cosmetic: the decklink_dec* files are used for input from the decklink cards. update the header to reflect that

cards. update the header to reflect that On Tue, Aug 02, 2016 at 06:44:14PM +, Felt, Patrick wrote: This is a cosmetic patch only to update the header of decklink files to be indicative of data flow direction. --- libavdevice/decklink_dec.cpp | 4 ++-- libavdevice/decklink_dec.h | 2 +- libavdev

[FFmpeg-devel] [PATCH] - cosmetic: the decklink_dec* files are used for input from the decklink cards. update the header to reflect that

This is a cosmetic patch only to update the header of decklink files to be indicative of data flow direction. --- libavdevice/decklink_dec.cpp | 4 ++-- libavdevice/decklink_dec.h | 2 +- libavdevice/decklink_dec_c.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] Remove last deprecated calls

l@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH] Remove last deprecated calls On Mon, 27 Jun 2016, Felt, Patrick wrote: --- libavdevice/decklink_dec.cpp | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/deckli

Re: [FFmpeg-devel] [PATCH] Remove last deprecated calls

Agreed. I was headed to fixing up the input mode autodetection next as that’s still outstanding in my queue. I wanted/needed to clean up those errors first though so i didn’t have to wrap those calls in FF_DISABLE_DEPRECATION_WARNINGS and subsequently forgetting to remove them when I submit

[FFmpeg-devel] [PATCH] Remove last deprecated calls

--- libavdevice/decklink_dec.cpp | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp index 7412727..9c5d5f9 100644 --- a/libavdevice/decklink_dec.cpp +++ b/libavdevice/decklink_dec.cpp @@ -120,10 +120,6

[FFmpeg-devel] [PATCH] Change the decklink_dec file header to reflect that this module is used for input from decklink and not output

--- libavdevice/decklink_dec.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp index fcb024e..7412727 100644 --- a/libavdevice/decklink_dec.cpp +++ b/libavdevice/decklink_dec.cpp @@ -1,5 +1,5 @@ /* - * Blackmagic

Re: [FFmpeg-devel] bans

On 6/15/16, 7:50 AM, "ffmpeg-devel on behalf of James Almer" wrote: >On 6/15/2016 10:14 AM, Michael Niedermayer wrote: >> After writing this mail i will >> >> 1. ban carl for 24h from the ML due to >> causing derek to leave the

Re: [FFmpeg-devel] IRC meeting

On 6/4/16, 4:33 AM, "ffmpeg-devel on behalf of Michael Niedermayer" wrote: >On Sat, Jun 04, 2016 at 10:30:32AM +0200, Piotr Bandurski wrote: >> Hi, >> >> The problem with this project is that it has not enough active

Re: [FFmpeg-devel] [PATCH] Updated (v3) -- Add input mode autodetect to the decklink module.

On 5/21/16, 3:38 AM, "ffmpeg-devel on behalf of Marton Balint" wrote: >> --- a/libavdevice/decklink_common.cpp >> +++ b/libavdevice/decklink_common.cpp >> @@ -98,6 +98,90 @@ HRESULT ff_decklink_get_display_name(IDeckLink *This, >>

Re: [FFmpeg-devel] Tee improvement - discussion

On 5/20/16, 9:32 PM, "ffmpeg-devel on behalf of Michael Niedermayer" wrote: >On Fri, May 20, 2016 at 12:06:34AM +0200, Marton Balint wrote: >> >> On Thu, 19 May 2016, Nicolas George wrote: >> >> >Le tridi 23 floréal, an

[FFmpeg-devel] FW: [PATCH] -- Add input swap functionality to movie filter(src_movie.c)

bump On 5/16/16, 2:15 PM, "ffmpeg-devel on behalf of Felt, Patrick" <ffmpeg-devel-boun...@ffmpeg.org on behalf of patrick.f...@echostar.com> wrote: >This is a rework of the previously submitted patch to not require globals. >I’ve also renamed variables per standard.

Re: [FFmpeg-devel] [PATCH] Updated (v3) -- Add input mode autodetect to the decklink module.

bump On 5/16/16, 9:28 AM, "ffmpeg-devel on behalf of Felt, Patrick" <ffmpeg-devel-boun...@ffmpeg.org on behalf of patrick.f...@echostar.com> wrote: >bump > >On 5/12/16, 4:07 PM, "ffmpeg-devel on behalf of Felt, Patrick" ><ffmpeg-devel-boun...@ffmpeg.o

Re: [FFmpeg-devel] [PATCH] Updated (v3) -- Add input mode autodetect to the decklink module.

bump On 5/16/16, 9:28 AM, "ffmpeg-devel on behalf of Felt, Patrick" <ffmpeg-devel-boun...@ffmpeg.org on behalf of patrick.f...@echostar.com> wrote: >bump > >On 5/12/16, 4:07 PM, "ffmpeg-devel on behalf of Felt, Patrick" ><ffmpeg-devel-boun...@ffmpeg.o

[FFmpeg-devel] [PATCH] -- Add input swap functionality to movie filter(src_movie.c)

This is a rework of the previously submitted patch to not require globals. I’ve also renamed variables per standard. Attached is the output of git format-patch (let me know if I should just paste contents into the body in the future. This just felt a little cleaner). swap-official-patch

Re: [FFmpeg-devel] [PATCH] Updated (v3) -- Add input mode autodetect to the decklink module.

bump On 5/12/16, 4:07 PM, "ffmpeg-devel on behalf of Felt, Patrick" <ffmpeg-devel-boun...@ffmpeg.org on behalf of patrick.f...@echostar.com> wrote: >I hang my head in shame. I neglected to notice that time wasn’t already >included and so I had to modify the patch. A

Re: [FFmpeg-devel] [PATCH] Updated -- Add input mode autodetect to the decklink module.

"all", NULL, 0, AV_OPT_TYPE_CONST, { .i64 = 0x7LL}, 0, 0,DEC, "teletext_lines"}, { "channels", "number of audio channels", OFFSET(audio_channels), AV_OPT_TYPE_INT , { .i64 = 2 }, 2,

Re: [FFmpeg-devel] [PATCH] -- Add input mode autodetect to the decklink module.

Ah. Didn’t realize we had that. Will do. On 5/12/16, 1:37 PM, "ffmpeg-devel on behalf of Marton Balint" <ffmpeg-devel-boun...@ffmpeg.org on behalf of c...@passwd.hu> wrote: > > >On Thu, 12 May 2016, Matthias Hunstock wrote: > >> Am 12.05.2016 um 19:16 schrieb

[FFmpeg-devel] [PATCH] -- Add input mode autodetect to the decklink module.

Previously users had to supply the input format like this 'DeckLink Device@modenum'. This patch allows users to either leave it off completely, or supply 0 or negative number to indicate autodetect is requested. Autodetect only works the first time so if the mode changes mid stream you'll

Re: [FFmpeg-devel] [PATCH] -- Convert decklink input module to use codecpar

Bump On 5/7/16, 10:02 PM, "ffmpeg-devel on behalf of Felt, Patrick" <ffmpeg-devel-boun...@ffmpeg.org on behalf of patrick.f...@echostar.com> wrote: >This patch converts decklink input to use codecpar. There are still a couple >of deprecated calls that I

[FFmpeg-devel] [PATCH] -- Convert decklink input module to use codecpar

This patch converts decklink input to use codecpar. There are still a couple of deprecated calls that I’m not sure what to do with. They are both related to some logic around AVCodecContext.coded_frame. I couldn’t find anywhere that really documented what that was, or where it was intended

Re: [FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

On 5/4/16, 9:12 AM, "ffmpeg-devel on behalf of Hendrik Leppkes" <ffmpeg-devel-boun...@ffmpeg.org on behalf of h.lepp...@gmail.com> wrote: >On Wed, May 4, 2016 at 5:01 PM, Felt, Patrick <patrick.f...@echostar.com> >wrote: >>> >>>[...] >>&g

Re: [FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

On 5/4/16, 5:52 AM, "ffmpeg-devel on behalf of Clément Bœsch" <ffmpeg-devel-boun...@ffmpeg.org on behalf of u...@pkh.me> wrote: >On Wed, May 04, 2016 at 12:15:26AM +, Felt, Patrick wrote: >> Afternoon all, >> I apologize if this isn’t right way to sub

Re: [FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

trying include it in the body On 5/3/16, 8:34 PM, "ffmpeg-devel on behalf of Michael Niedermayer" <ffmpeg-devel-boun...@ffmpeg.org on behalf of mich...@niedermayer.cc> wrote: >On Wed, May 04, 2016 at 12:15:26AM +, Felt, Patrick wrote: >> Afternoon all, >>

[FFmpeg-devel] [Patch] Add input swap functionality to movie filter (src_movie.c)

Afternoon all, I apologize if this isn’t right way to submit a patch. Attached is a patch for src_movie.c that modifies it to allow for one to use the process_command() infrastructure to swap the input file on the fly. I’ve added a few options to the filter and exposed filename to the