moved fallback APE/ID3 tag loader to song.c

Some plugins used the APE or ID3 tag loader as a fallback when their
own methods of loading tags did not work.  Move this code out of all
decoder plugins, into song_file_update().

5 files changed, 19 insertions, 57 deletions

diff --git a/src/decoder/aac_plugin.c b/src/decoder/aac_plugin.c
index f2907129e..90d39113b 100644
--- a/src/decoder/aac_plugin.c
+++ b/src/decoder/aac_plugin.c
@@ -443,19 +443,18 @@ aac_stream_decode(struct decoder *mpd_decoder, struct input_stream *inStream)
 
 static struct tag *aacTagDup(const char *file)
 {
-	struct tag *ret = NULL;
 	int file_time = getAacTotalTime(file);
+	struct tag *tag;
 
-	if (file_time >= 0) {
-		if ((ret = tag_id3_load(file)) == NULL)
-			ret = tag_new();
-		ret->time = file_time;
-	} else {
+	if (file_time < 0) {
 		g_debug("aacTagDup: Failed to get total song time from: %s\n",
 			file);
+		return NULL;
 	}
 
-	return ret;
+	tag = tag_new();
+	tag->time = file_time;
+	return tag;
 }
 
 static const char *const aac_suffixes[] = { "aac", NULL };
diff --git a/src/decoder/flac_plugin.c b/src/decoder/flac_plugin.c
index e02c13638..19f8d7fd4 100644
--- a/src/decoder/flac_plugin.c
+++ b/src/decoder/flac_plugin.c
@@ -282,23 +282,7 @@ flac_tag_load(const char *file)
 static struct tag *
 flac_tag_dup(const char *file)
 {
-	struct tag *ret = NULL;
-
-	ret = flac_tag_load(file);
-	if (!ret) {
-		g_debug("Failed to grab information from: %s\n", file);
-		return NULL;
-	}
-	if (tag_is_empty(ret)) {
-		struct tag *temp = tag_id3_load(file);
-		if (temp) {
-			temp->time = ret->time;
-			tag_free(ret);
-			ret = temp;
-		}
-	}
-
-	return ret;
+	return flac_tag_load(file);
 }
 
 static void
diff --git a/src/decoder/mp3_plugin.c b/src/decoder/mp3_plugin.c
index 3444a7e94..fd0ac21a9 100644
--- a/src/decoder/mp3_plugin.c
+++ b/src/decoder/mp3_plugin.c
@@ -19,6 +19,7 @@
 #include "../decoder_api.h"
 #include "../conf.h"
 #include "config.h"
+#include "tag_id3.h"
 
 #include <assert.h>
 #include <unistd.h>
@@ -1190,22 +1191,19 @@ mp3_decode(struct decoder *decoder, struct input_stream *input_stream)
 
 static struct tag *mp3_tag_dup(const char *file)
 {
-	struct tag *ret = NULL;
+	struct tag *tag;
 	int total_time;
 
-	ret = tag_id3_load(file);
-
 	total_time = mp3_total_file_time(file);
-	if (total_time >= 0) {
-		if (!ret)
-			ret = tag_new();
-		ret->time = total_time;
-	} else {
+	if (total_time < 0) {
 		g_debug("mp3_tag_dup: Failed to get total song time from: %s\n",
 			file);
+		return NULL;
 	}
 
-	return ret;
+	tag = tag_new();
+	tag->time = total_time;
+	return tag;
 }
 
 static const char *const mp3_suffixes[] = { "mp3", "mp2", NULL };
diff --git a/src/decoder/mp4_plugin.c b/src/decoder/mp4_plugin.c
index bc36acf8a..fd7415137 100644
--- a/src/decoder/mp4_plugin.c
+++ b/src/decoder/mp4_plugin.c
@@ -405,21 +405,7 @@ mp4_load_tag(const char *file)
 static struct tag *
 mp4_tag_dup(const char *file)
 {
-	struct tag *ret = NULL;
-
-	ret = mp4_load_tag(file);
-	if (!ret)
-		return NULL;
-	if (tag_is_empty(ret)) {
-		struct tag *temp = tag_id3_load(file);
-		if (temp) {
-			temp->time = ret->time;
-			tag_free(ret);
-			ret = temp;
-		}
-	}
-
-	return ret;
+	return mp4_load_tag(file);
 }
 
 static const char *const mp4_suffixes[] = { "m4a", "mp4", NULL };
diff --git a/src/decoder/mpc_plugin.c b/src/decoder/mpc_plugin.c
index 5ef2c5448..23fa5787b 100644
--- a/src/decoder/mpc_plugin.c
+++ b/src/decoder/mpc_plugin.c
@@ -249,8 +249,8 @@ static float mpcGetTime(const char *file)
 
 static struct tag *mpcTagDup(const char *file)
 {
-	struct tag *ret = NULL;
 	float total_time = mpcGetTime(file);
+	struct tag *tag;
 
 	if (total_time < 0) {
 		g_debug("mpcTagDup: Failed to get Songlength of file: %s\n",
@@ -258,14 +258,9 @@ static struct tag *mpcTagDup(const char *file)
 		return NULL;
 	}
 
-	ret = tag_ape_load(file);
-	if (!ret)
-		ret = tag_id3_load(file);
-	if (!ret)
-		ret = tag_new();
-	ret->time = total_time;
-
-	return ret;
+	tag = tag_new();
+	tag->time = total_time;
+	return tag;
 }
 
 static const char *const mpcSuffixes[] = { "mpc", NULL };