mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-01-03 05:10:03 +02:00
libfdk-aac: Consistently use a proper version check macro for detecting features
The previous version checks checked explicitly for the version where the version define was added to the installed headers, making an "#ifdef AACDECODER_LIB_VL0" enough. Now that we have a need for more diverse version checks than this, convert all checks to such checks. Signed-off-by: Martin Storsjö <martin@martin.st>
This commit is contained in:
parent
642fd4769b
commit
ffb9b7a6ba
@ -25,9 +25,12 @@
|
||||
#include "avcodec.h"
|
||||
#include "internal.h"
|
||||
|
||||
/* The version macro is introduced the same time as the setting enum was
|
||||
* changed, so this check should suffice. */
|
||||
#ifndef AACDECODER_LIB_VL0
|
||||
#define FDKDEC_VER_AT_LEAST(vl0, vl1) \
|
||||
(defined(AACDECODER_LIB_VL0) && \
|
||||
((AACDECODER_LIB_VL0 > vl0) || \
|
||||
(AACDECODER_LIB_VL0 == vl0 && AACDECODER_LIB_VL1 >= vl1)))
|
||||
|
||||
#if !FDKDEC_VER_AT_LEAST(2, 5) // < 2.5.10
|
||||
#define AAC_PCM_MAX_OUTPUT_CHANNELS AAC_PCM_OUTPUT_CHANNELS
|
||||
#endif
|
||||
|
||||
@ -72,7 +75,7 @@ static const AVOption fdk_aac_dec_options[] = {
|
||||
OFFSET(drc_level), AV_OPT_TYPE_INT, { .i64 = -1}, -1, 127, AD, NULL },
|
||||
{ "drc_heavy", "Dynamic Range Control: heavy compression, where [1] is on (RF mode) and [0] is off",
|
||||
OFFSET(drc_heavy), AV_OPT_TYPE_INT, { .i64 = -1}, -1, 1, AD, NULL },
|
||||
#ifdef AACDECODER_LIB_VL0
|
||||
#if FDKDEC_VER_AT_LEAST(2, 5) // 2.5.10
|
||||
{ "level_limit", "Signal level limiting", OFFSET(level_limit), AV_OPT_TYPE_INT, { .i64 = 0 }, -1, 1, AD },
|
||||
#endif
|
||||
{ NULL }
|
||||
@ -296,7 +299,7 @@ static av_cold int fdk_aac_decode_init(AVCodecContext *avctx)
|
||||
}
|
||||
}
|
||||
|
||||
#ifdef AACDECODER_LIB_VL0
|
||||
#if FDKDEC_VER_AT_LEAST(2, 5) // 2.5.10
|
||||
if (aacDecoder_SetParam(s->handle, AAC_PCM_LIMITER_ENABLE, s->level_limit) != AAC_DEC_OK) {
|
||||
av_log(avctx, AV_LOG_ERROR, "Unable to set in signal level limiting in the decoder\n");
|
||||
return AVERROR_UNKNOWN;
|
||||
|
@ -159,7 +159,7 @@ static av_cold int aac_encode_init(AVCodecContext *avctx)
|
||||
case 6: mode = MODE_1_2_2_1; sce = 2; cpe = 2; break;
|
||||
/* The version macro is introduced the same time as the 7.1 support, so this
|
||||
should suffice. */
|
||||
#ifdef AACENCODER_LIB_VL0
|
||||
#if FDKENC_VER_AT_LEAST(3, 4) // 3.4.12
|
||||
case 8:
|
||||
sce = 2;
|
||||
cpe = 3;
|
||||
@ -295,7 +295,7 @@ static av_cold int aac_encode_init(AVCodecContext *avctx)
|
||||
}
|
||||
|
||||
avctx->frame_size = info.frameLength;
|
||||
#if FDKENC_VER_AT_LEAST(4, 0)
|
||||
#if FDKENC_VER_AT_LEAST(4, 0) // 4.0.0
|
||||
avctx->initial_padding = info.nDelay;
|
||||
#else
|
||||
avctx->initial_padding = info.encoderDelay;
|
||||
@ -418,7 +418,7 @@ static const uint64_t aac_channel_layout[] = {
|
||||
AV_CH_LAYOUT_4POINT0,
|
||||
AV_CH_LAYOUT_5POINT0_BACK,
|
||||
AV_CH_LAYOUT_5POINT1_BACK,
|
||||
#ifdef AACENCODER_LIB_VL0
|
||||
#if FDKENC_VER_AT_LEAST(3, 4) // 3.4.12
|
||||
AV_CH_LAYOUT_7POINT1_WIDE_BACK,
|
||||
AV_CH_LAYOUT_7POINT1,
|
||||
#endif
|
||||
|
Loading…
Reference in New Issue
Block a user