aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-02-18 18:39:12 +0100
committerMax Kellermann <max@duempel.org>2009-02-18 18:39:12 +0100
commit51c59f622872c7fb028c121cc74300de34a80783 (patch)
treee624baa4142fd9bb323544b58949bf2b56591186
parent111c73e701e61316c7aef41bf4330759cfaba5e7 (diff)
downloadmpd-51c59f622872c7fb028c121cc74300de34a80783.tar.gz
mpd-51c59f622872c7fb028c121cc74300de34a80783.tar.xz
mpd-51c59f622872c7fb028c121cc74300de34a80783.zip
mp4ff: use faacDecInit2() to find the AAC track
Use faacDecInit2() instead of AudioSpecificConfig() to detect the AAC track in the MP4 file. This has a great advantage: it initializes the libfaad decoder, which the caller would normally do anyway - but now we can go without the AudioSpecificConfig() call. When decoder==NULL (called from mp4_tag_dup()), fall back to a mp4ff_get_track_type()==1 check, like other audio players do.
Diffstat (limited to '')
-rw-r--r--configure.ac1
-rw-r--r--src/decoder/mp4ff_plugin.c85
2 files changed, 36 insertions, 50 deletions
diff --git a/configure.ac b/configure.ac
index 61c821604..3b60f4b51 100644
--- a/configure.ac
+++ b/configure.ac
@@ -812,7 +812,6 @@ int main() {
])
fi
if test x$enable_aac = xyes; then
- AC_CHECK_TYPES(mp4AudioSpecificConfig,,,[#include <faad.h>])
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
diff --git a/src/decoder/mp4ff_plugin.c b/src/decoder/mp4ff_plugin.c
index fb0f00776..db9632f4d 100644
--- a/src/decoder/mp4ff_plugin.c
+++ b/src/decoder/mp4ff_plugin.c
@@ -38,39 +38,45 @@ struct mp4_context {
};
static int
-mp4_get_aac_track(mp4ff_t * infile)
+mp4_get_aac_track(mp4ff_t * infile, faacDecHandle decoder,
+ uint32_t *sample_rate, unsigned char *channels_r)
{
- /* find AAC track */
+#ifdef HAVE_FAAD_LONG
+ /* neaacdec.h declares all arguments as "unsigned long", but
+ internally expects uint32_t pointers. To avoid gcc
+ warnings, use this workaround. */
+ unsigned long *sample_rate_r = (unsigned long*)sample_rate;
+#else
+ uint32_t *sample_rate_r = sample_rate;
+#endif
int i, rc;
int num_tracks = mp4ff_total_tracks(infile);
for (i = 0; i < num_tracks; i++) {
unsigned char *buff = NULL;
unsigned int buff_size = 0;
-#ifdef HAVE_MP4AUDIOSPECIFICCONFIG
- mp4AudioSpecificConfig mp4ASC;
-#else
- unsigned long dummy1_32;
- unsigned char dummy2_8, dummy3_8, dummy4_8, dummy5_8, dummy6_8,
- dummy7_8, dummy8_8;
-#endif
+
+ if (mp4ff_get_track_type(infile, i) != 1)
+ /* not an audio track */
+ continue;
+
+ if (decoder == NULL)
+ /* have don't have a decoder to initialize -
+ we're done now, because we found an audio
+ track */
+ return i;
mp4ff_get_decoder_config(infile, i, &buff, &buff_size);
+ if (buff == NULL)
+ continue;
- if (buff) {
-#ifdef HAVE_MP4AUDIOSPECIFICCONFIG
- rc = AudioSpecificConfig(buff, buff_size, &mp4ASC);
-#else
- rc = AudioSpecificConfig(buff, &dummy1_32, &dummy2_8,
- &dummy3_8, &dummy4_8,
- &dummy5_8, &dummy6_8,
- &dummy7_8, &dummy8_8);
-#endif
- free(buff);
- if (rc < 0)
- continue;
+ rc = faacDecInit2(decoder, buff, buff_size,
+ sample_rate_r, channels_r);
+ free(buff);
+
+ if (rc >= 0)
+ /* found a valid AAC track */
return i;
- }
}
/* can't decode this */
@@ -95,21 +101,12 @@ mp4_seek(void *user_data, uint64_t position)
}
static faacDecHandle
-mp4_faad_new(mp4ff_t *mp4fh, int track, struct audio_format *audio_format)
+mp4_faad_new(mp4ff_t *mp4fh, int *track_r, struct audio_format *audio_format)
{
faacDecHandle decoder;
faacDecConfigurationPtr config;
- unsigned char *mp4_buffer;
- unsigned int mp4_buffer_size;
+ int track;
uint32_t sample_rate;
-#ifdef HAVE_FAAD_LONG
- /* neaacdec.h declares all arguments as "unsigned long", but
- internally expects uint32_t pointers. To avoid gcc
- warnings, use this workaround. */
- unsigned long *sample_rate_r = (unsigned long*)&sample_rate;
-#else
- uint32_t *sample_rate_r = &sample_rate;
-#endif
unsigned char channels;
decoder = faacDecOpen();
@@ -124,17 +121,14 @@ mp4_faad_new(mp4ff_t *mp4fh, int track, struct audio_format *audio_format)
#endif
faacDecSetConfiguration(decoder, config);
- mp4_buffer = NULL;
- mp4_buffer_size = 0;
- mp4ff_get_decoder_config(mp4fh, track, &mp4_buffer, &mp4_buffer_size);
-
- if (faacDecInit2(decoder, mp4_buffer, mp4_buffer_size,
- sample_rate_r, &channels) < 0) {
- g_warning("Not an AAC stream.\n");
+ track = mp4_get_aac_track(mp4fh, decoder, &sample_rate, &channels);
+ if (track < 0) {
+ g_warning("No AAC track found");
faacDecClose(decoder);
return NULL;
}
+ *track_r = track;
*audio_format = (struct audio_format){
.bits = 16,
.channels = channels,
@@ -196,14 +190,7 @@ mp4_decode(struct decoder *mpd_decoder, struct input_stream *input_stream)
return;
}
- track = mp4_get_aac_track(mp4fh);
- if (track < 0) {
- g_warning("No AAC track found in mp4 stream.\n");
- mp4ff_close(mp4fh);
- return;
- }
-
- decoder = mp4_faad_new(mp4fh, track, &audio_format);
+ decoder = mp4_faad_new(mp4fh, &track, &audio_format);
if (decoder == NULL) {
mp4ff_close(mp4fh);
return;
@@ -374,7 +361,7 @@ mp4_tag_dup(const char *file)
return NULL;
}
- track = mp4_get_aac_track(mp4fh);
+ track = mp4_get_aac_track(mp4fh, NULL, NULL, NULL);
if (track < 0) {
mp4ff_close(mp4fh);
input_stream_close(&input_stream);