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 f
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 patches
Subject: [FFmpeg-devel] RFQ on patch for decklink warnings
Afternoon all,
I’m lo
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 a/libavdevi
:08 PM
To: FFmpeg development discussions and patches
Subject: Re: [FFmpeg-devel] [PATCH] - cosmetic: the decklink_dec* files are
used for input from the decklink 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
---
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
___
ent discussions and patches
Date: Tuesday, August 2, 2016 at 11:49 PM
To: FFmpeg development discussions and patches
Subject: Re: [FFmpeg-devel] [PATCH] - libavdevices/decklink_*.cpp: formatting
change to remove uneeded spaces in initializers
On Tue, Aug 2, 2016 at 10:17 PM Felt, Patr
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 process
trying include it in the body
On 5/3/16, 8:34 PM, "ffmpeg-devel on behalf of Michael Niedermayer"
wrote:
>On Wed, May 04, 2016 at 12:15:26AM +, Felt, Patrick wrote:
>> Afternoon all,
>> I apologize if this isn’t right way to submit a patch. Attached is a patch
&
On 5/4/16, 5:52 AM, "ffmpeg-devel on behalf of Clément Bœsch"
wrote:
>On Wed, May 04, 2016 at 12:15:26AM +, Felt, Patrick wrote:
>> Afternoon all,
>> I apologize if this isn’t right way to submit a patch.
>
>You're supposed to git commit and use g
On 5/4/16, 9:12 AM, "ffmpeg-devel on behalf of Hendrik Leppkes"
wrote:
>On Wed, May 4, 2016 at 5:01 PM, Felt, Patrick
>wrote:
>>>
>>>[...]
>>>> +/* libavfilter documentation says that filter init will be called only
>>>> o
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 t
Bump
On 5/7/16, 10:02 PM, "ffmpeg-devel on behalf of Felt, Patrick"
wrote:
>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 AVCodecConte
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
die.
Ah. Didn’t realize we had that. Will do.
On 5/12/16, 1:37 PM, "ffmpeg-devel on behalf of Marton Balint"
wrote:
>
>
>On Thu, 12 May 2016, Matthias Hunstock wrote:
>
>> Am 12.05.2016 um 19:16 schrieb Felt, Patrick:
>>> +while (!ctx->video)
Ah… I see a bug. I should either be setting ctx->video in the if block or I
should be breaking instead of continuing. Good catch!
On 5/12/16, 1:32 PM, "ffmpeg-devel on behalf of Matthias Hunstock"
wrote:
>Am 12.05.2016 um 19:16 schrieb Felt, Patrick:
>> +
@@ static const AVOption options[] = {
{ "standard", NULL, 0,
AV_OPT_TYPE_CONST, { .i64 = 0x7fff9fffeLL}, 0, 0,DEC, "teletext_lines"},
{ "all", NULL, 0,
AV_OPT
@ -36,6 +36,7 @@ static const AVOption options[] = {
{ "standard", NULL, 0,
AV_OPT_TYPE_CONST, { .i64 = 0x7fff9fffeLL}, 0, 0,DEC, "teletext_lines"},
{ "all", NULL, 0,
bump
On 5/12/16, 4:07 PM, "ffmpeg-devel on behalf of Felt, Patrick"
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. Apologies for the noise.
>
>-- Add input mode autodetect to the decklink mo
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
bump
On 5/16/16, 9:28 AM, "ffmpeg-devel on behalf of Felt, Patrick"
wrote:
>bump
>
>On 5/12/16, 4:07 PM, "ffmpeg-devel on behalf of Felt, Patrick"
> wrote:
>
>>I hang my head in shame. I neglected to notice that time wasn’t already
>>included a
bump
On 5/16/16, 9:28 AM, "ffmpeg-devel on behalf of Felt, Patrick"
wrote:
>bump
>
>On 5/12/16, 4:07 PM, "ffmpeg-devel on behalf of Felt, Patrick"
> wrote:
>
>>I hang my head in shame. I neglected to notice that time wasn’t already
>>included a
bump
On 5/16/16, 2:15 PM, "ffmpeg-devel on behalf of Felt, Patrick"
wrote:
>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
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 CCXXIV, Jan Sebechlebsky a écrit :
>> >>My current idea is to create que
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,
>> const char **displayName)
>> return hr;
>> }
>>
>> +long ff_
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 developers.
>> It looks like most of the devs swiched into "stand-by" mode
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 project. (24h was suggested in the IRC
>> meeting)
>
>This is us
---
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 DeckL
---
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 @@
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 my
From: ffmpeg-devel on behalf of Marton Balint
Reply-To: FFmpeg development discussions and patches
Date: Monday, June 27, 2016 at 6:01 PM
To: FFmpeg development discussions and patches
Subject: Re: [FFmpeg-devel] [PATCH] Remove last deprecated calls
On Mon, 27 Jun 2016, Felt, Patrick
30 matches
Mail list logo
