[FFmpeg-devel] avdevice/decklink_dec: fix build with older SDK

Apparently bmdFormatUnspecified needs SDK 11.0. It is just a fancy way of
checking for zero, so let's do that instead.

Fixes build issue since f1b908d20a8.

Signed-off-by: Marton Balint <cus@passwd.hu>
---
 libavdevice/decklink_dec.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

On 15-09-2020 12:54 am, Marton Balint wrote:
> Apparently bmdFormatUnspecified needs SDK 11.0. It is just a fancy way of
> checking for zero, so let's do that instead.
>
> Fixes build issue since f1b908d20a8.
>
> Signed-off-by: Marton Balint <cus@passwd.hu>
> ---
>   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 92b8feed14..6517b9df13 100644
> --- a/libavdevice/decklink_dec.cpp
> +++ b/libavdevice/decklink_dec.cpp
> @@ -1232,7 +1232,7 @@ av_cold int ff_decklink_read_header(AVFormatContext *avctx)
>           }
>           av_log(avctx, AV_LOG_INFO, "Autodetected the input mode\n");
>       }
> -    if (ctx->raw_format == bmdFormatUnspecified)
> +    if (ctx->raw_format == (BMDPixelFormat)0)
>           ctx->raw_format = bmdFormat8BitYUV;
>       if (ff_decklink_set_format(avctx, DIRECTION_IN) < 0) {
>           av_log(avctx, AV_LOG_ERROR, "Could not set format code %s for %s\n",

LGTM.

Gyan

On Tue, 15 Sep 2020, Gyan Doshi wrote:

>
>
> On 15-09-2020 12:54 am, Marton Balint wrote:
>> Apparently bmdFormatUnspecified needs SDK 11.0. It is just a fancy way of
>> checking for zero, so let's do that instead.
>>
>> Fixes build issue since f1b908d20a8.
>>
>> Signed-off-by: Marton Balint <cus@passwd.hu>
>> ---
>>   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 92b8feed14..6517b9df13 100644
>> --- a/libavdevice/decklink_dec.cpp
>> +++ b/libavdevice/decklink_dec.cpp
>> @@ -1232,7 +1232,7 @@ av_cold int ff_decklink_read_header(AVFormatContext 
> *avctx)
>>           }
>>           av_log(avctx, AV_LOG_INFO, "Autodetected the input mode\n");
>>       }
>> -    if (ctx->raw_format == bmdFormatUnspecified)
>> +    if (ctx->raw_format == (BMDPixelFormat)0)
>>           ctx->raw_format = bmdFormat8BitYUV;
>>       if (ff_decklink_set_format(avctx, DIRECTION_IN) < 0) {
>>           av_log(avctx, AV_LOG_ERROR, "Could not set format code %s for 
> %s\n",
>
> LGTM.

Thanks, applied.

Regards,
Marton