[FFmpeg-devel] [PATCH 1/8] libavdevice/decklink: Add support for EIA-708 output over SDI
Carl Eugen Hoyos
ceffmpeg at gmail.com
Fri Dec 29 22:41:30 EET 2017
2017-12-29 19:12 GMT+01:00 Devin Heitmueller <dheitmueller at ltnglobal.com>:
> + side_data = av_frame_get_side_data(pic, AV_FRAME_DATA_A53_CC);
> + if (side_data && side_data->size) {
> + uint8_t *buf = av_packet_new_side_data(pkt, AV_PKT_DATA_A53_CC, side_data->size);
> + if (buf)
> + memcpy(buf, side_data->data, side_data->size);
> + else
> + return AVERROR(ENOMEM);
Maybe you disagree but the following is slightly simpler imo:
if (!buf)
return AVERROR();
memcpy(buf, data, size);
[...]
> +#if CONFIG_LIBKLVANC
I tried to voice this before and I assume there is no solution
but this is a large optional code block depending on an
external library inside an optional feature depending on
another - not super-common - external library: This will not
get much testing, not even for compilation.
Carl Eugen
More information about the ffmpeg-devel
mailing list