[FFmpeg-devel] [PATCH 2/2] lavd: add decklink output device

Ramiro Polla ramiro.polla at gmail.com
Sat Dec 28 20:06:16 CET 2013


Hi Reimar,

On Sat, Dec 28, 2013 at 1:23 PM, Reimar Döffinger
<Reimar.Doeffinger at gmx.de> wrote:
> On 26.12.2013, at 02:44, Ramiro Polla <ramiro.polla at gmail.com> wrote:
>> Attached patch adds decklink output support to lavd. Works under Linux
>> and Windows.
>
> Maybe a bit crazy, but how about splitting the C++ part out even more?
> So that we can use designated initializers like for all other demuxers.
> Otherwise it will break as soon as we add a new field in the middle, and in addition unless you add FATE testing that works without the hardware/library "nobody" will even notice.

I thought about so too. On Windows it is possible to use COBJMACROS
and write the code in C. But for Linux only a C++ interface is
provided. What do you think about putting just the AVOption, AVClass
and AVOutputFormat in a C file?

> (of course if people just stopped using C++ when it makes 0 sense like for a trivial library interface that would avoid that and many more issues...).

Ramiro


More information about the ffmpeg-devel mailing list