aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2013-01-29 16:17:15 +0100
committerMax Kellermann <max@duempel.org>2013-01-29 16:17:15 +0100
commit3dd8beb3809d9c61d21e12e7422cca3cbb2afc59 (patch)
treec226da9d95a0f9f4ecfed0dc5d0aca9354dea35e
parentae77406b8ebfa3462f776f2d0b9bf4ee0ca36d8d (diff)
downloadmpd-3dd8beb3809d9c61d21e12e7422cca3cbb2afc59.tar.gz
mpd-3dd8beb3809d9c61d21e12e7422cca3cbb2afc59.tar.xz
mpd-3dd8beb3809d9c61d21e12e7422cca3cbb2afc59.zip
decoder/faad: use the newer NeAAC* API
Drop support for the old faacDec* API.
-rw-r--r--m4/faad.m465
-rw-r--r--src/decoder/faad_decoder_plugin.c67
2 files changed, 38 insertions, 94 deletions
diff --git a/m4/faad.m4 b/m4/faad.m4
index cda4298e3..5ca520e79 100644
--- a/m4/faad.m4
+++ b/m4/faad.m4
@@ -23,77 +23,36 @@ if test x$enable_aac = xyes; then
AC_CHECK_DECL(FAAD2_VERSION,,enable_aac=no,[#include <faad.h>])
fi
if test x$enable_aac = xyes; then
- AC_CHECK_DECL(faacDecInit2,,enable_aac=no,[#include <faad.h>])
+ AC_CHECK_LIB(faad,NeAACDecInit2,,enable_aac=no)
fi
if test x$enable_aac = xyes; then
- AC_CHECK_LIB(faad,faacDecInit2,,enable_aac=no)
- if test x$enable_aac = xno; then
- enable_aac=yes
- AC_CHECK_LIB(faad,NeAACDecInit2,,enable_aac=no)
- fi
- fi
- if test x$enable_aac = xyes; then
- AC_MSG_CHECKING(that FAAD2 uses buffer and bufferlen)
- AC_COMPILE_IFELSE([AC_LANG_SOURCE([
-#include <faad.h>
-
-int main() {
- char buffer;
- long bufferlen = 0;
- faacDecHandle decoder;
- faacDecFrameInfo frameInfo;
- faacDecConfigurationPtr config;
- unsigned char channels;
- long sampleRate;
- mp4AudioSpecificConfig mp4ASC;
-
- decoder = faacDecOpen();
- config = faacDecGetCurrentConfiguration(decoder);
- config->outputFormat = FAAD_FMT_16BIT;
- faacDecSetConfiguration(decoder,config);
- AudioSpecificConfig(&buffer, bufferlen, &mp4ASC);
- faacDecInit(decoder,&buffer,bufferlen,&sampleRate,&channels);
- faacDecInit2(decoder,&buffer,bufferlen,&sampleRate,&channels);
- faacDecDecode(decoder,&frameInfo,&buffer,bufferlen);
-
- return 0;
-}
-])],[AC_MSG_RESULT(yes);AC_DEFINE(HAVE_FAAD_BUFLEN_FUNCS,1,[Define if FAAD2 uses buflen in function calls])],[AC_MSG_RESULT(no);
AC_MSG_CHECKING(that FAAD2 can even be used)
AC_COMPILE_IFELSE([AC_LANG_SOURCE([
#include <faad.h>
int main() {
char buffer;
- faacDecHandle decoder;
- faacDecFrameInfo frameInfo;
- faacDecConfigurationPtr config;
+ NeAACDecHandle decoder;
+ NeAACDecFrameInfo frameInfo;
+ NeAACDecConfigurationPtr config;
unsigned char channels;
long sampleRate;
long bufferlen = 0;
- unsigned long dummy1_32;
- unsigned char dummy2_8, dummy3_8, dummy4_8, dummy5_8, dummy6_8,
- dummy7_8, dummy8_8;
- decoder = faacDecOpen();
- config = faacDecGetCurrentConfiguration(decoder);
+ decoder = NeAACDecOpen();
+ config = NeAACDecGetCurrentConfiguration(decoder);
config->outputFormat = FAAD_FMT_16BIT;
- faacDecSetConfiguration(decoder,config);
- AudioSpecificConfig(&buffer,&dummy1_32,&dummy2_8,
- &dummy3_8,&dummy4_8,&dummy5_8,
- &dummy6_8,&dummy7_8,&dummy8_8);
- faacDecInit(decoder,&buffer,&sampleRate,&channels);
- faacDecInit2(decoder,&buffer,bufferlen,&sampleRate,&channels);
- faacDecDecode(decoder,&frameInfo,&buffer);
- faacDecClose(decoder);
+ NeAACDecSetConfiguration(decoder,config);
+ NeAACDecInit(decoder,&buffer,bufferlen,&sampleRate,&channels);
+ NeAACDecInit2(decoder,&buffer,bufferlen,&sampleRate,&channels);
+ NeAACDecDecode(decoder,&frameInfo,&buffer,bufferlen);
+ NeAACDecClose(decoder);
return 0;
}
])],AC_MSG_RESULT(yes),[AC_MSG_RESULT(no);enable_aac=no])
- ])
fi
if test x$enable_aac = xyes; then
- AC_CHECK_MEMBERS([faacDecConfiguration.downMatrix,faacDecConfiguration.dontUpSampleImplicitSBR,faacDecFrameInfo.samplerate],,,[#include <faad.h>])
AC_DEFINE(HAVE_FAAD,1,[Define to use FAAD2 for AAC decoding])
else
AC_MSG_WARN([faad2 lib needed for MP4/AAC support -- disabling MP4/AAC support])
@@ -121,7 +80,7 @@ int main() {
unsigned char channels;
uint32_t sample_rate;
- faacDecInit2(NULL, NULL, 0, &sample_rate, &channels);
+ NeAACDecInit2(NULL, NULL, 0, &sample_rate, &channels);
return 0;
}
])],
diff --git a/src/decoder/faad_decoder_plugin.c b/src/decoder/faad_decoder_plugin.c
index c8dbb966c..a7ece93fe 100644
--- a/src/decoder/faad_decoder_plugin.c
+++ b/src/decoder/faad_decoder_plugin.c
@@ -23,16 +23,18 @@
#include "audio_check.h"
#include "tag_handler.h"
-#define AAC_MAX_CHANNELS 6
+#include <neaacdec.h>
+
+#include <glib.h>
#include <assert.h>
#include <unistd.h>
-#include <faad.h>
-#include <glib.h>
#undef G_LOG_DOMAIN
#define G_LOG_DOMAIN "faad"
+#define AAC_MAX_CHANNELS 6
+
static const unsigned adts_sample_rates[] =
{ 96000, 88200, 64000, 48000, 44100, 32000, 24000, 22050,
16000, 12000, 11025, 8000, 7350, 0, 0, 0
@@ -239,11 +241,11 @@ faad_song_duration(struct decoder_buffer *buffer, struct input_stream *is)
}
/**
- * Wrapper for faacDecInit() which works around some API
+ * Wrapper for NeAACDecInit() which works around some API
* inconsistencies in libfaad.
*/
static bool
-faad_decoder_init(faacDecHandle decoder, struct decoder_buffer *buffer,
+faad_decoder_init(NeAACDecHandle decoder, struct decoder_buffer *buffer,
struct audio_format *audio_format, GError **error_r)
{
union {
@@ -271,10 +273,8 @@ faad_decoder_init(faacDecHandle decoder, struct decoder_buffer *buffer,
return false;
}
- nbytes = faacDecInit(decoder, u.out,
-#ifdef HAVE_FAAD_BUFLEN_FUNCS
+ nbytes = NeAACDecInit(decoder, u.out,
length,
-#endif
sample_rate_p, &channels);
if (nbytes < 0) {
g_set_error(error_r, faad_decoder_quark(), 0,
@@ -289,12 +289,12 @@ faad_decoder_init(faacDecHandle decoder, struct decoder_buffer *buffer,
}
/**
- * Wrapper for faacDecDecode() which works around some API
+ * Wrapper for NeAACDecDecode() which works around some API
* inconsistencies in libfaad.
*/
static const void *
-faad_decoder_decode(faacDecHandle decoder, struct decoder_buffer *buffer,
- faacDecFrameInfo *frame_info)
+faad_decoder_decode(NeAACDecHandle decoder, struct decoder_buffer *buffer,
+ NeAACDecFrameInfo *frame_info)
{
union {
/* deconst hack for libfaad */
@@ -302,20 +302,13 @@ faad_decoder_decode(faacDecHandle decoder, struct decoder_buffer *buffer,
void *out;
} u;
size_t length;
- void *result;
u.in = decoder_buffer_read(buffer, &length);
if (u.in == NULL)
return NULL;
- result = faacDecDecode(decoder, frame_info,
- u.out
-#ifdef HAVE_FAAD_BUFLEN_FUNCS
- , length
-#endif
- );
-
- return result;
+ return NeAACDecDecode(decoder, frame_info,
+ u.out, length);
}
/**
@@ -328,8 +321,6 @@ faad_get_file_time_float(struct input_stream *is)
{
struct decoder_buffer *buffer;
float length;
- faacDecHandle decoder;
- faacDecConfigurationPtr config;
buffer = decoder_buffer_new(NULL, is,
FAAD_MIN_STREAMSIZE * AAC_MAX_CHANNELS);
@@ -339,11 +330,12 @@ faad_get_file_time_float(struct input_stream *is)
bool ret;
struct audio_format audio_format;
- decoder = faacDecOpen();
+ NeAACDecHandle decoder = NeAACDecOpen();
- config = faacDecGetCurrentConfiguration(decoder);
+ NeAACDecConfigurationPtr config =
+ NeAACDecGetCurrentConfiguration(decoder);
config->outputFormat = FAAD_FMT_16BIT;
- faacDecSetConfiguration(decoder, config);
+ NeAACDecSetConfiguration(decoder, config);
decoder_buffer_fill(buffer);
@@ -351,7 +343,7 @@ faad_get_file_time_float(struct input_stream *is)
if (ret)
length = 0;
- faacDecClose(decoder);
+ NeAACDecClose(decoder);
}
decoder_buffer_free(buffer);
@@ -381,9 +373,7 @@ faad_stream_decode(struct decoder *mpd_decoder, struct input_stream *is)
{
GError *error = NULL;
float total_time = 0;
- faacDecHandle decoder;
struct audio_format audio_format;
- faacDecConfigurationPtr config;
bool ret;
uint16_t bit_rate = 0;
struct decoder_buffer *buffer;
@@ -395,17 +385,14 @@ faad_stream_decode(struct decoder *mpd_decoder, struct input_stream *is)
/* create the libfaad decoder */
- decoder = faacDecOpen();
+ NeAACDecHandle decoder = NeAACDecOpen();
- config = faacDecGetCurrentConfiguration(decoder);
+ NeAACDecConfigurationPtr config =
+ NeAACDecGetCurrentConfiguration(decoder);
config->outputFormat = FAAD_FMT_16BIT;
-#ifdef HAVE_FAACDECCONFIGURATION_DOWNMATRIX
config->downMatrix = 1;
-#endif
-#ifdef HAVE_FAACDECCONFIGURATION_DONTUPSAMPLEIMPLICITSBR
config->dontUpSampleImplicitSBR = 0;
-#endif
- faacDecSetConfiguration(decoder, config);
+ NeAACDecSetConfiguration(decoder, config);
while (!decoder_buffer_is_full(buffer) &&
!input_stream_lock_eof(is) &&
@@ -420,7 +407,7 @@ faad_stream_decode(struct decoder *mpd_decoder, struct input_stream *is)
if (!ret) {
g_warning("%s", error->message);
g_error_free(error);
- faacDecClose(decoder);
+ NeAACDecClose(decoder);
return;
}
@@ -433,7 +420,7 @@ faad_stream_decode(struct decoder *mpd_decoder, struct input_stream *is)
do {
size_t frame_size;
const void *decoded;
- faacDecFrameInfo frame_info;
+ NeAACDecFrameInfo frame_info;
/* find the next frame */
@@ -448,7 +435,7 @@ faad_stream_decode(struct decoder *mpd_decoder, struct input_stream *is)
if (frame_info.error > 0) {
g_warning("error decoding AAC stream: %s\n",
- faacDecGetErrorMessage(frame_info.error));
+ NeAACDecGetErrorMessage(frame_info.error));
break;
}
@@ -458,14 +445,12 @@ faad_stream_decode(struct decoder *mpd_decoder, struct input_stream *is)
break;
}
-#ifdef HAVE_FAACDECFRAMEINFO_SAMPLERATE
if (frame_info.samplerate != audio_format.sample_rate) {
g_warning("sample rate changed from %u to %lu",
audio_format.sample_rate,
(unsigned long)frame_info.samplerate);
break;
}
-#endif
decoder_buffer_consume(buffer, frame_info.bytesconsumed);
@@ -486,7 +471,7 @@ faad_stream_decode(struct decoder *mpd_decoder, struct input_stream *is)
/* cleanup */
- faacDecClose(decoder);
+ NeAACDecClose(decoder);
}
static bool