Skip to content

Commit

Permalink
Fix Building Against Newer Version Of FFmpeg
Browse files Browse the repository at this point in the history
When building against a newer version:
* Use `ch_layout.order` & `ch_layout.nb_channels` instead of `channel_layout` & `channels`.
*  Workaround `read_probe` no longer being exposed by calling `av_probe_input_format`
  • Loading branch information
CuriousTommy committed Aug 18, 2024
1 parent 203af1f commit 8d475d8
Show file tree
Hide file tree
Showing 2 changed files with 38 additions and 8 deletions.
12 changes: 8 additions & 4 deletions src/CoreAudio/AFAVFormatComponent/AudioFileFormatGeneric.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -81,10 +81,6 @@ void AudioFileFormatGeneric::GetFileTypeName(CFStringRef *outName)

UncertainResult AudioFileFormatGeneric::FileDataIsThisFormat(UInt32 inDataByteSize, const void* inData)
{
const AVInputFormat* fmt = av_find_input_format(m_avformatShortName);
if (!fmt)
return false;

std::vector<uint8_t> buf;
AVProbeData probeData;

Expand All @@ -96,7 +92,15 @@ UncertainResult AudioFileFormatGeneric::FileDataIsThisFormat(UInt32 inDataByteSi
probeData.buf = buf.data();
probeData.buf_size = inDataByteSize;

#warning "TODO: Remove old `read_probe` call once we no longer support older distros"
#if LIBAVCODEC_VERSION_MAJOR >= 61
return av_probe_input_format(&probeData, false) != nullptr ? kTrue : kFalse;
#else
const AVInputFormat* fmt = av_find_input_format(m_avformatShortName);
if (!fmt)
return false;
return fmt->read_probe(&probeData) ? kTrue : kFalse;
#endif
}

AudioFileObject* AudioFileFormatGeneric::New()
Expand Down
34 changes: 30 additions & 4 deletions src/CoreAudio/AudioToolbox/AudioConverterImpl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -100,10 +100,19 @@ OSStatus AudioConverter::create(const AudioStreamBasicDescription* inSourceForma

if (inSourceFormat->mFormatID == kAudioFormatLinearPCM)
{
#warning "TODO: Remove deprecated 'channels' once we no longer support older distros"
#if LIBAVCODEC_VERSION_MAJOR >= 61
cIn->ch_layout.nb_channels = inSourceFormat->mChannelsPerFrame;
#else
cIn->channels = inSourceFormat->mChannelsPerFrame;
#endif
cIn->sample_rate = inSourceFormat->mSampleRate;


#if LIBAVCODEC_VERSION_MAJOR >= 61
std::cout << "Converting from PCM with " << cIn->ch_layout.nb_channels << " channels at " << cIn->sample_rate << " Hz\n";
#else
std::cout << "Converting from PCM with " << cIn->channels << " channels at " << cIn->sample_rate << " Hz\n";
#endif
}

if (avcodec_open2((*out)->m_decoder, codecIn, nullptr) < 0)
Expand Down Expand Up @@ -131,9 +140,15 @@ void AudioConverter::initEncoder()

m_encoder->codec_type = AVMEDIA_TYPE_AUDIO;
m_encoder->bit_rate = m_outBitRate;
#warning "TODO: Remove deprecated 'channels' once we no longer support older distros"
#if LIBAVCODEC_VERSION_MAJOR >= 61
m_encoder->ch_layout.order = AV_CHANNEL_ORDER_UNSPEC;
m_encoder->ch_layout.nb_channels = m_destinationFormat.mChannelsPerFrame;
#else
m_encoder->channels = m_destinationFormat.mChannelsPerFrame;
m_encoder->sample_rate = m_destinationFormat.mSampleRate;
m_encoder->channel_layout = CAChannelCountToLayout(m_destinationFormat.mChannelsPerFrame);
#endif
m_encoder->sample_rate = m_destinationFormat.mSampleRate;
m_encoder->sample_fmt = CACodecSampleFormat(&m_destinationFormat);

#ifdef DEBUG_AUDIOCONVERTER
Expand Down Expand Up @@ -161,10 +176,16 @@ void AudioConverter::allocateBuffers()

m_audioFrame->nb_samples = ENCODER_FRAME_SAMPLES;
m_audioFrame->format = m_encoder->sample_fmt;
m_audioFrame->channel_layout = m_encoder->channel_layout;


#warning "TODO: Remove deprecated 'channels' once we no longer support older distros"
#if LIBAVCODEC_VERSION_MAJOR >= 61
m_audioFrame->ch_layout.order = m_encoder->ch_layout.order;
m_audioFrame->ch_layout.nb_channels = m_encoder->ch_layout.nb_channels;
int audioSampleBuffer_size = av_samples_get_buffer_size(nullptr, m_encoder->ch_layout.nb_channels, m_audioFrame->nb_samples, m_encoder->sample_fmt, 0);
#else
m_audioFrame->channel_layout = m_encoder->channel_layout;
int audioSampleBuffer_size = av_samples_get_buffer_size(nullptr, m_encoder->channels, m_audioFrame->nb_samples, m_encoder->sample_fmt, 0);
#endif
void* audioSampleBuffer = (uint8_t*) av_malloc(audioSampleBuffer_size);

if (!audioSampleBuffer)
Expand All @@ -174,8 +195,13 @@ void AudioConverter::allocateBuffers()
}

// Setup the data pointers in the AVFrame
#if LIBAVCODEC_VERSION_MAJOR >= 61
if (int err = avcodec_fill_audio_frame(m_audioFrame, m_encoder->ch_layout.nb_channels, m_encoder->sample_fmt,
(const uint8_t*) audioSampleBuffer, audioSampleBuffer_size, 0 ); err < 0)
#else
if (int err = avcodec_fill_audio_frame(m_audioFrame, m_encoder->channels, m_encoder->sample_fmt,
(const uint8_t*) audioSampleBuffer, audioSampleBuffer_size, 0 ); err < 0)
#endif
{
std::cerr << "AudioConverter::allocateBuffers(): Could not set up audio frame\n";
throw std::runtime_error("AudioConverter::allocateBuffers(): Could not set up audio frame");
Expand Down

0 comments on commit 8d475d8

Please sign in to comment.