From: Andre Noll <maan@systemlinux.org>
Date: Sat, 24 Mar 2007 21:22:31 +0000 (+0100)
Subject: mp3_afh.c: set bitrate, frequency and channels of struct audio_file_info
X-Git-Tag: v0.2.16~7^2~3
X-Git-Url: http://git.tue.mpg.de/?a=commitdiff_plain;h=e402aa45bbac0b078fefc88ed6dd108cbae6e743;p=paraslash.git

mp3_afh.c: set bitrate, frequency and channels of struct audio_file_info

Also, get rid of some superflous includes.
---

diff --git a/mp3_afh.c b/mp3_afh.c
index cf507ec2..b510e72f 100644
--- a/mp3_afh.c
+++ b/mp3_afh.c
@@ -28,9 +28,7 @@
  *			 Johannes Overmann <overmann@iname.com>
  */
 
-#include "server.cmdline.h"
 #include "server.h"
-#include "vss.h"
 #include "error.h"
 #include "fd.h"
 #include "string.h"
@@ -72,8 +70,6 @@ struct mp3info {
 	int id3_isvalid;
 	struct id3tag id3;
 	int vbr;
-	long unsigned br_average;
-	int freq;
 };
 
 /** \endcond */
@@ -115,6 +111,16 @@ static const char *header_mode(struct mp3header *h)
 		h->mode = 4; /* invalid */
 	return mode_text[h->mode];
 }
+
+static int header_channels(struct mp3header *h)
+{
+	if (h->mode > 3)
+		return 0;
+	if (h->mode < 3)
+		return 2;
+	return 1;
+}
+
 static int header_bitrate(struct mp3header *h)
 {
 	if (h->layer > 3 || h->bitrate > 14)
@@ -143,14 +149,14 @@ static void write_info_str(struct audio_format_info *afi)
 	int v = mp3.id3_isvalid;
 
 	snprintf(afi->info_string, MMD_INFO_SIZE,
-		"audio_file_info1:%lu x %lums, %lu kbit/s (%cbr) %i KHz %s\n"
+		"audio_file_info1:%lu x %lums, %u kbit/s (%cbr) %i KHz %s\n"
 		"audio_file_info2:%s, by %s\n"
 		"audio_file_info3:A: %s, Y: %s, C: %s\n",
 		afi->chunks_total,
 		tv2ms(&afi->chunk_tv),
-		mp3.br_average,
+		afi->bitrate,
 		mp3.vbr? 'v' : 'c',
-		mp3.freq / 1000,
+		afi->frequency / 1000,
 		header_mode(&mp3.header),
 		v && *mp3.id3.title? mp3.id3.title : "(title tag not set)",
 		v && *mp3.id3.artist? mp3.id3.artist : "(artist tag not set)",
@@ -334,7 +340,6 @@ static int mp3_read_info(unsigned char *map, off_t numbytes,
 	mp3_get_id3(map, numbytes, &fpos);
 	fpos = 0;
 	mp3.vbr = 0;
-	mp3.freq = 0;
 	while (1) {
 		int freq, br, fl;
 		struct timeval tmp, cct; /* current chunk time */
@@ -381,8 +386,9 @@ static int mp3_read_info(unsigned char *map, off_t numbytes,
 	if (!afi->chunks_total || !freq_avg || !br_avg)
 		goto err_out;
 	afi->chunk_table[afi->chunks_total] = numbytes - 1;
-	mp3.br_average = br_avg;
-	mp3.freq = freq_avg;
+	afi->bitrate = br_avg;
+	afi->frequency = freq_avg;
+	afi->channels = header_channels(&mp3.header);
 	afi->seconds_total = (tv2ms(&total_time) + 500) / 1000;
 	tv_divide(afi->chunks_total, &total_time, &afi->chunk_tv);
 	PARA_DEBUG_LOG("%lu chunks, each %lums\n", afi->chunks_total,