X-Git-Url: http://git.tuebingen.mpg.de/?a=blobdiff_plain;f=afh_common.c;h=75d8b5118db91586d46f339fff0d1d1f693dfc07;hb=234647bb5139513bdf36c1afec46f5c43ba27adc;hp=063ae8f0b104ccab15e9f10f637ec34e7783e785;hpb=9d8fdf8898418711558f70a9630b2444036d8285;p=paraslash.git

diff --git a/afh_common.c b/afh_common.c
index 063ae8f0..75d8b511 100644
--- a/afh_common.c
+++ b/afh_common.c
@@ -139,6 +139,38 @@ int guess_audio_format(const char *name)
 	return -E_AUDIO_FORMAT;
 }
 
+/**
+ * Get the name of the given audio format.
+ *
+ * \param i The audio format number.
+ *
+ * \return This returns a pointer to statically allocated memory so it
+ * must not be freed by the caller.
+ */
+const char *audio_format_name(int i)
+{
+	if (i < 0 || i >= ARRAY_SIZE(afl) - 1)
+		return "???";
+	return afl[i].name;
+}
+
+static int get_file_info(int format, const char *path, char *data,
+		size_t size, int fd, struct afh_info *afhi)
+{
+	int ret;
+	const char *fmt = audio_format_name(format);
+
+	memset(afhi, 0, sizeof(*afhi));
+	ret = afl[format].get_file_info(data, size, fd, afhi);
+	if (ret < 0) {
+		PARA_WARNING_LOG("%s: %s format not detected: %s\n",
+			path, fmt, para_strerror(-ret));
+		return ret;
+	}
+	PARA_NOTICE_LOG("%s: detected %s format\n", path, fmt);
+	return format;
+}
+
 /**
  * Call get_file_info() to obtain an afhi structure.
  *
@@ -163,31 +195,18 @@ int compute_afhi(const char *path, char *data, size_t size, int fd,
 {
 	int ret, i, format;
 
-	afhi->header_len = 0;
-	afhi->techinfo = NULL;
-	afhi->tags.artist = NULL;
-	afhi->tags.title = NULL;
-	afhi->tags.year = NULL;
-	afhi->tags.album = NULL;
-	afhi->tags.comment = NULL;
 	format = guess_audio_format(path);
-
 	if (format >= 0) {
-		ret = afl[format].get_file_info(data, size, fd, afhi);
-		if (ret >= 0) {
-			ret = format;
+		ret = get_file_info(format, path, data, size, fd, afhi);
+		if (ret >= 0)
 			goto success;
-		}
 	}
 	FOR_EACH_AUDIO_FORMAT(i) {
 		if (i == format) /* we already tried this one to no avail */
 			continue;
-		ret = afl[i].get_file_info(data, size, fd, afhi);
-		if (ret >= 0) {
-			ret = i;
+		ret = get_file_info(i, path, data, size, fd, afhi);
+		if (ret >= 0)
 			goto success;
-		}
-		PARA_WARNING_LOG("%s\n", para_strerror(-ret));
 	}
 	return -E_AUDIO_FORMAT;
 success:
@@ -233,21 +252,6 @@ void clear_afhi(struct afh_info *afhi)
 	free(afhi->tags.comment);
 }
 
-/**
- * Get the name of the given audio format.
- *
- * \param i The audio format number.
- *
- * \return This returns a pointer to statically allocated memory so it
- * must not be freed by the caller.
- */
-const char *audio_format_name(int i)
-{
-	if (i < 0 || i >= ARRAY_SIZE(afl) - 1)
-		return "???";
-	return afl[i].name;
-}
-
 static inline size_t get_chunk_len(long unsigned chunk_num,
 		const struct afh_info *afhi)
 {
@@ -291,8 +295,8 @@ int32_t afh_get_start_chunk(int32_t approx_chunk_num,
 
 	for (k = PARA_MAX(0, approx_chunk_num); k >= 0; k--)
 		if (get_chunk_len(k, afhi) > 0)
-			break;
-	return k;
+			return k;
+	return 0;
 }
 
 /**
@@ -370,6 +374,7 @@ unsigned afh_get_afhi_txt(int audio_format_num, struct afh_info *afhi, char **re
 		"%s: %" PRIu32 "\n" /* seconds total */
 		"%s: %lu: %lu\n" /* chunk time */
 		"%s: %" PRIu32 "\n" /* num chunks */
+		"%s: %" PRIu32 "\n" /* max chunk size */
 		"%s: %s\n" /* techinfo */
 		"%s: %s\n" /* artist */
 		"%s: %s\n" /* title */
@@ -384,6 +389,7 @@ unsigned afh_get_afhi_txt(int audio_format_num, struct afh_info *afhi, char **re
 		status_item_list[SI_CHUNK_TIME], (long unsigned)afhi->chunk_tv.tv_sec,
 			(long unsigned)afhi->chunk_tv.tv_usec,
 		status_item_list[SI_NUM_CHUNKS], afhi->chunks_total,
+		status_item_list[SI_MAX_CHUNK_SIZE], afhi->max_chunk_size,
 		status_item_list[SI_TECHINFO], afhi->techinfo? afhi->techinfo : "",
 		status_item_list[SI_ARTIST], afhi->tags.artist? afhi->tags.artist : "",
 		status_item_list[SI_TITLE], afhi->tags.title? afhi->tags.title : "",
@@ -393,6 +399,37 @@ unsigned afh_get_afhi_txt(int audio_format_num, struct afh_info *afhi, char **re
 	);
 }
 
+/**
+ * Determine the maximal chunk size by investigating the chunk table.
+ *
+ * \param afhi Value/result.
+ *
+ * This function iterates over the chunk table and sets ->max_chunk_size
+ * accordingly. The function exists only for backward compatibility since as of
+ * version 0.6.0, para_server stores the maximal chunk size in its database.
+ * This function is only called if the database value is zero, indicating that
+ * the file was added by an older server version.
+ */
+void set_max_chunk_size(struct afh_info *afhi)
+{
+	uint32_t n, max = 0, old = 0;
+
+	for (n = 0; n <= afhi->chunks_total; n++) {
+		uint32_t val = afhi->chunk_table[n];
+		/*
+		 * If the first chunk is the header, do not consider it for the
+		 * calculation of the largest chunk size.
+		 */
+		if (n == 0 || (n == 1 && afhi->header_len > 0)) {
+			old = val;
+			continue;
+		}
+		max = PARA_MAX(max, val - old);
+		old = val;
+	}
+	afhi->max_chunk_size = max;
+}
+
 /**
  * Create a copy of the given file with altered meta tags.
  *