From: Andre <maan@p133.(none)>
Date: Mon, 17 Apr 2006 18:44:22 +0000 (+0200)
Subject: play.c: reorder and rename some functions.
X-Git-Tag: v0.2.12~80
X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=commitdiff_plain;h=301d17664161885d5e92d18404d53862de3c77f8;hp=d6aab2f273e8b54bf351c3b73b4a50168c8be917

play.c: reorder and rename some functions.

Move everything which depends on ALSA to the three functions
alsa_init(), former set_alsa_params(), alsa_write(), former
pcm_write(), and alsa_shutdown().
---

diff --git a/play.c b/play.c
index ad522a95..f459e4dd 100644
--- a/play.c
+++ b/play.c
@@ -61,8 +61,8 @@ do { if (EXP) \
 while (0)
 
 static snd_pcm_t *handle;
-static unsigned char *audiobuf;
 static snd_pcm_uframes_t chunk_size;
+static unsigned char *audiobuf;
 static size_t bytes_per_frame;
 static struct timeval *start_time;
 static struct gengetopt_args_info conf;
@@ -94,11 +94,11 @@ static void read_wav_header(void)
 }
 
 /*
- * set_alsa_params - Prepare the PCM handle for writing
+ * open and prepare the PCM handle for writing
  *
  * Install PCM software and hardware configuration. Exit on errors.
  */
-static void set_alsa_params(void)
+static void alsa_init(void)
 {
 	snd_pcm_hw_params_t *hwparams;
 	snd_pcm_sw_params_t *swparams;
@@ -106,6 +106,18 @@ static void set_alsa_params(void)
 		stop_threshold;
 	unsigned buffer_time = 0;
 	int err;
+	snd_pcm_info_t *info;
+	snd_output_t *log;
+
+	snd_pcm_info_alloca(&info);
+	if (snd_output_stdio_attach(&log, stderr, 0) < 0)
+		EXIT(E_LOG);
+	err = snd_pcm_open(&handle, conf.device_arg,
+		SND_PCM_STREAM_PLAYBACK, 0);
+	if (err < 0)
+		EXIT(E_PCM_OPEN);
+	if ((err = snd_pcm_info(handle, info)) < 0)
+		EXIT(E_SND_PCM_INFO);
 
 	snd_pcm_hw_params_alloca(&hwparams);
 	snd_pcm_sw_params_alloca(&swparams);
@@ -159,14 +171,14 @@ static void set_alsa_params(void)
 	bytes_per_frame = snd_pcm_format_physical_width(FORMAT) * conf.channels_arg / 8;
 }
 
-/*
- * pcm_write - push out pcm frames
+/**
+ * push out pcm frames
  * \param data pointer do data to be written
  * \param count number of frames
  *
  * \return Number of bytes written. Exit on errors.
  */
-static snd_pcm_sframes_t pcm_write(u_char *data, size_t count)
+int alsa_write(u_char *data, size_t count)
 {
 	snd_pcm_sframes_t r, result = 0;
 	while (count > 0) {
@@ -186,7 +198,14 @@ static snd_pcm_sframes_t pcm_write(u_char *data, size_t count)
 			data += r * bytes_per_frame;
 		}
 	}
-	return result;
+	return result * bytes_per_frame;
+}
+
+void alsa_shutdown(void)
+{
+	snd_pcm_drain(handle);
+	snd_pcm_close(handle);
+	snd_config_update_free_global();
 }
 
 /*
@@ -239,7 +258,7 @@ static void play_pcm(size_t loaded)
 	unsigned char *p;
 	struct timeval delay;
 
-	set_alsa_params();
+	alsa_init();
 	chunk_bytes = chunk_size * bytes_per_frame;
 	bufsize = (conf.bufsize_arg * 1024 / chunk_bytes) * chunk_bytes;
 	audiobuf = realloc(audiobuf, bufsize);
@@ -257,7 +276,7 @@ again:
 	}
 	p = audiobuf;
 	while (loaded >= chunk_bytes) {
-		ret = pcm_write(p, chunk_size) * bytes_per_frame;
+		ret = alsa_write(p, chunk_size);
 		p += ret;
 		written += ret;
 		loaded -= ret;
@@ -272,7 +291,7 @@ read:
 		loaded += ret;
 		goto again;
 	}
-	snd_pcm_drain(handle);
+	alsa_shutdown();
 }
 
 /*
@@ -293,10 +312,7 @@ static size_t check_wave(void)
 
 int main(int argc, char *argv[])
 {
-	snd_pcm_info_t *info;
-	snd_output_t *log;
 	struct timeval tv;
-	int err;
 
 	cmdline_parser(argc, argv, &conf);
 	if (conf.start_time_given) {
@@ -305,21 +321,9 @@ int main(int argc, char *argv[])
 			EXIT(E_SYNTAX);
 		start_time = &tv;
 	}
-	snd_pcm_info_alloca(&info);
-	if (snd_output_stdio_attach(&log, stderr, 0) < 0)
-		EXIT(E_LOG);
-	err = snd_pcm_open(&handle, conf.device_arg,
-		SND_PCM_STREAM_PLAYBACK, 0);
-	if (err < 0)
-		EXIT(E_PCM_OPEN);
-	if ((err = snd_pcm_info(handle, info)) < 0)
-		EXIT(E_SND_PCM_INFO);
 	audiobuf = malloc(WAV_HEADER_LEN);
 	read_wav_header();
 	play_pcm(check_wave());
-	snd_pcm_close(handle);
 	free(audiobuf);
-//	snd_output_close(log);
-	snd_config_update_free_global();
 	return EXIT_SUCCESS;
 }